Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22570 )
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 initialization 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 Reviewed-on: http://gerrit.cloudera.org:8080/22570 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Riza Suminto <[email protected]> --- M tests/common/custom_cluster_test_suite.py M tests/custom_cluster/test_query_live.py 2 files changed, 17 insertions(+), 29 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved Riza Suminto: Verified -- 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: merged Gerrit-Change-Id: Ieb4c86fa79bb1df000b6241bdd31c7641d807c4f Gerrit-Change-Number: 22570 Gerrit-PatchSet: 8 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]>
