Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22210 )

Change subject: IMPALA-13603: Fix Flaky Live Queries Table Tests
......................................................................

IMPALA-13603: Fix Flaky Live Queries Table Tests

Workload management skips recording successful trivial DDLs but does
not skip recording failed trivial DDLs. The test_query_live.py tests
run a describe DDL in the setup_method(). Normally, this describe
succeeds immediately and thus is not recorded in the workload
management tables. However, in very rare instances, the describe DDL
will fail the first time and succeed the second time. These cases
result in an extra query recorded in the sys.impala_query_live table
and test assertions that rely on a certain number of records being in
this table fail because there are extra records.

This change modifies the method used to determine if the
sys.impala_query_live table is available. The test_query_live.py
tests now check the coordinator's catalog cache and wait until the
sys.impala_query_live table appears. DDLs are no longer executed.

All tests in the test_query_live.py file passed locally and in
Jenkins builds.

Change-Id: I767a2bd7b068ab3fdeddb7cb3c4b307844d2d279
Reviewed-on: http://gerrit.cloudera.org:8080/22210
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M tests/custom_cluster/test_query_live.py
1 file changed, 9 insertions(+), 2 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/22210
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I767a2bd7b068ab3fdeddb7cb3c4b307844d2d279
Gerrit-Change-Number: 22210
Gerrit-PatchSet: 8
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to