Hello Riza Suminto, Pranav Lodha, Michael Smith, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/22210

to look at the new patch set (#5).

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
suceedes 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
---
M tests/custom_cluster/test_query_live.py
1 file changed, 9 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/22210/5
--
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: newpatchset
Gerrit-Change-Id: I767a2bd7b068ab3fdeddb7cb3c4b307844d2d279
Gerrit-Change-Number: 22210
Gerrit-PatchSet: 5
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