Hello Tim Armstrong, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9538 to look at the new patch set (#2). Change subject: IMPALA-6602: fixes flaky expiration test ...................................................................... IMPALA-6602: fixes flaky expiration test The test_query_expiration test assumes that a metric and the query state are maintained atomically. Since they're not, occasionaly flakes (false negatives) occur. The fix in this patch is to loop until the expected state is seen. If the expected state is not seen with a given number of iterations, the test fails. These tests depend on timing so if this validation takes too long, the test will also fail. Such looping is used in the two places where its assumed that the client state and metrics are maintained atomically. Change-Id: I7aabed87d84d5cfd8078cc6c39df48e22ff30afc --- M tests/custom_cluster/test_query_expiration.py 1 file changed, 18 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/38/9538/2 -- To view, visit http://gerrit.cloudera.org:8080/9538 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I7aabed87d84d5cfd8078cc6c39df48e22ff30afc Gerrit-Change-Number: 9538 Gerrit-PatchSet: 2 Gerrit-Owner: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>