Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23081 )
Change subject: IMPALA-14164: Fix timeout for fragments in flight in TestScratchDir ...................................................................... IMPALA-14164: Fix timeout for fragments in flight in TestScratchDir On release builds, some tests in TestScratchDir have started hitting a timeout waiting for num-fragments-in-flight to reach 2. The code to wait for the metric sleeps one second between samples. If one of the query fragments starts and finishes during that second, the test will never see a sample containing two in-flight fragments. This happens on release builds because they are faster and more likely to complete within that second. This removes the code that waits for num-fragments-in-flight. All the tests have subsequent calls waiting for the scratch usage to reach a certain value. This will properly wait for the fragment to start up on its own. The num-fragments-in-flight wait doesn't add anything. Testing: - Ran custom_cluster/test_scratch_disk.py multiple times with a release build Change-Id: Ic8c573affc033056ba465c42bd420d5c1d3ba15c Reviewed-on: http://gerrit.cloudera.org:8080/23081 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M tests/custom_cluster/test_scratch_disk.py 1 file changed, 0 insertions(+), 19 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23081 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic8c573affc033056ba465c42bd420d5c1d3ba15c Gerrit-Change-Number: 23081 Gerrit-PatchSet: 4 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]>
