Hello Henry Robinson,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4325
to look at the new patch set (#2).
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
---
M tests/query_test/test_lifecycle.py
M tests/verifiers/metric_verifier.py
2 files changed, 5 insertions(+), 18 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/4325/2
--
To view, visit http://gerrit.cloudera.org:8080/4325
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I112e502a25f075928b0f6ef376c7fd9c6376ef4d
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
