-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-7356][STREAMING] Fix flakey tests in FlumePollingStreamSuite using SparkSink's batch CountDownLatch. #5918
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…sing SparkSink's batch CountDownLatch.
|
Merged build triggered. |
|
Merged build started. |
|
Test build #31899 has started for PR 5918 at commit |
|
Test build #31899 has finished for PR 5918 at commit
|
|
Merged build finished. Test PASSed. |
|
Test PASSed. |
|
Jenkins, test this please |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #31917 has started for PR 5918 at commit |
|
Test build #31917 has finished for PR 5918 at commit
|
|
Merged build finished. Test FAILed. |
|
Test FAILed. |
|
This test passed, unresolved deps in some other test caused the failure. |
|
Jenkins, test this please |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #31925 has started for PR 5918 at commit |
|
Test build #31925 has finished for PR 5918 at commit
|
|
Merged build finished. Test PASSed. |
|
Test PASSed. |
|
Ping @tdas |
|
test this please |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32011 has started for PR 5918 at commit |
|
Test build #32011 has finished for PR 5918 at commit
|
|
Merged build finished. Test FAILed. |
|
Test FAILed. |
|
Sorry, clicked the wrong button :/ |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32419 has started for PR 5918 at commit |
|
Test build #32419 has finished for PR 5918 at commit
|
|
Merged build finished. Test FAILed. |
|
Test FAILed. |
|
Unrelated failure in |
|
Jenkins, test this please |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32427 has started for PR 5918 at commit |
|
Test build #32427 has finished for PR 5918 at commit
|
|
Merged build finished. Test PASSed. |
|
Test PASSed. |
|
Jenkins, test this please |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32446 has started for PR 5918 at commit |
|
Test build #32446 has finished for PR 5918 at commit
|
|
Merged build finished. Test PASSed. |
|
Test PASSed. |
|
I took a quick look and it looks reasonable. I'm going to merge this into master and 1.4 first and we'll see if it fails again. Thanks @harishreedharan. |
…using SparkSink's batch CountDownLatch. This is meant to make the FlumePollingStreamSuite deterministic. Now we basically count the number of batches that have been completed - and then verify the results rather than sleeping for random periods of time. Author: Hari Shreedharan <[email protected]> Closes #5918 from harishreedharan/flume-test-fix and squashes the following commits: 93f24f3 [Hari Shreedharan] Add an eventually block to ensure that all received data is processed. Refactor the dstream creation and remove redundant code. 1108804 [Hari Shreedharan] [SPARK-7356][STREAMING] Fix flakey tests in FlumePollingStreamSuite using SparkSink's batch CountDownLatch. (cherry picked from commit 61d1e87) Signed-off-by: Andrew Or <[email protected]>
…using SparkSink's batch CountDownLatch. This is meant to make the FlumePollingStreamSuite deterministic. Now we basically count the number of batches that have been completed - and then verify the results rather than sleeping for random periods of time. Author: Hari Shreedharan <[email protected]> Closes apache#5918 from harishreedharan/flume-test-fix and squashes the following commits: 93f24f3 [Hari Shreedharan] Add an eventually block to ensure that all received data is processed. Refactor the dstream creation and remove redundant code. 1108804 [Hari Shreedharan] [SPARK-7356][STREAMING] Fix flakey tests in FlumePollingStreamSuite using SparkSink's batch CountDownLatch.
…using SparkSink's batch CountDownLatch. This is meant to make the FlumePollingStreamSuite deterministic. Now we basically count the number of batches that have been completed - and then verify the results rather than sleeping for random periods of time. Author: Hari Shreedharan <[email protected]> Closes apache#5918 from harishreedharan/flume-test-fix and squashes the following commits: 93f24f3 [Hari Shreedharan] Add an eventually block to ensure that all received data is processed. Refactor the dstream creation and remove redundant code. 1108804 [Hari Shreedharan] [SPARK-7356][STREAMING] Fix flakey tests in FlumePollingStreamSuite using SparkSink's batch CountDownLatch.
…using SparkSink's batch CountDownLatch. This is meant to make the FlumePollingStreamSuite deterministic. Now we basically count the number of batches that have been completed - and then verify the results rather than sleeping for random periods of time. Author: Hari Shreedharan <[email protected]> Closes apache#5918 from harishreedharan/flume-test-fix and squashes the following commits: 93f24f3 [Hari Shreedharan] Add an eventually block to ensure that all received data is processed. Refactor the dstream creation and remove redundant code. 1108804 [Hari Shreedharan] [SPARK-7356][STREAMING] Fix flakey tests in FlumePollingStreamSuite using SparkSink's batch CountDownLatch.
This is meant to make the FlumePollingStreamSuite deterministic. Now we basically count the number of batches that have been completed - and then verify the results rather than sleeping for random periods of time.