Hello Riza Suminto, Abhishek Rawat, gaurav singh, Michael Smith, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22570
to look at the new patch set (#5).
Change subject: IMPALA-13815: Fix Flaky Workload Management Tests
......................................................................
IMPALA-13815: Fix Flaky Workload Management Tests
The CustomClusterTestSuite.wait_for_wm_init() function checks for two
specific log lines to be logged by the catalog. The first line is
logged when workload manangement initialization is complete. The
second line is when a catalog topic update has been assembled.
However, if workload management initializtion is slow, then there may
not be a catalog topic update assembled after the initialization
completes. When this happens, an assertion fails despite the workload
management tables having been properly initialized and loaded by the
catalog.
This patch simplifies the CustomClusterTestSuite.wait_for_wm_init()
function so it waits until the catalogd logs it has completed
workload management initialization and then checks each coordinator's
local catalog cache for the workload management tables.
The following test suites passed locally and in an ASAN build. These
tests all call the 'wait_for_wm_init' function of
CustomClusterTestSuite.
* tests/custom_cluster/test_query_live.py
* tests/custom_cluster/test_query_log.py
* tests/custom_cluster/test_workload_mgmt_init.py
* tests/custom_cluster/test_workload_mgmt_sql_details.py
Change-Id: Ieb4c86fa79bb1df000b6241bdd31c7641d807c4f
---
M tests/common/custom_cluster_test_suite.py
M tests/custom_cluster/test_query_live.py
2 files changed, 17 insertions(+), 29 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/22570/5
--
To view, visit http://gerrit.cloudera.org:8080/22570
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ieb4c86fa79bb1df000b6241bdd31c7641d807c4f
Gerrit-Change-Number: 22570
Gerrit-PatchSet: 5
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: gaurav singh <[email protected]>
