Hello Michael Ho, Thomas Tauber-Marshall, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9590 to look at the new patch set (#2). Change subject: IMPALA-6498: test_query_profile_thrift_timestamps causes following tests to fail. ...................................................................... IMPALA-6498: test_query_profile_thrift_timestamps causes following tests to fail. test_query_profile_thrift_timestamps uses ImapaTestSuite.client.close() to force cancellation/unregistration of the query, so that 'EndTime' of the query shows up in the profile. Since other test cases also need a valid ImpalaTestSuite.client, we move the test case in question to its own test suite. Have also reduced the query to 'select sleep(5)', as the earlier 'select sleep(10000)' is just really excessively long. Change-Id: I287a05f3c90b1a71a5b7ee0f5c06a8840ebac4c9 --- M tests/query_test/test_observability.py A tests/query_test/test_thrift_profile.py 2 files changed, 82 insertions(+), 49 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/90/9590/2 -- To view, visit http://gerrit.cloudera.org:8080/9590 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I287a05f3c90b1a71a5b7ee0f5c06a8840ebac4c9 Gerrit-Change-Number: 9590 Gerrit-PatchSet: 2 Gerrit-Owner: Zoram Thanga <zo...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>