Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4087: TestFragmentLifecycle.test_failure_in_prepare ......................................................................
IMPALA-4087: TestFragmentLifecycle.test_failure_in_prepare The test previously got the "initial" value of number of fragments by reading the metric. This gave an incorrect non-zero result if there were any leftover queries running on the cluster. Avoid the problem and simplify the test by explicitly waiting for the number of fragments to go to zero. Change-Id: I112e502a25f075928b0f6ef376c7fd9c6376ef4d Reviewed-on: http://gerrit.cloudera.org:8080/4325 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Internal Jenkins --- M tests/query_test/test_lifecycle.py M tests/verifiers/metric_verifier.py 2 files changed, 5 insertions(+), 18 deletions(-) Approvals: Internal Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4325 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I112e502a25f075928b0f6ef376c7fd9c6376ef4d Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Tim Armstrong <[email protected]>
