This is an automated email from the ASF dual-hosted git repository. stigahuang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 0816986b15e299b227a1ae460d641111f75a78dd Author: Riza Suminto <[email protected]> AuthorDate: Tue Apr 22 13:54:21 2025 -0700 IMPALA-13987: Fix stress_catalog_init_delay_ms check in RELEASE stress_catalog_init_delay_ms does not exist in RELEASE build and causing KeyError in impala_cluster.py. This patch fix it by specifying default value when inspecting ImpaladService.get_flag_current_values() return value. Testing: Run start-impala-cluster.py in RELEASE build and it works. Change-Id: Ia4400a7e711d21d23cc37878f18f2e0389b741b0 Reviewed-on: http://gerrit.cloudera.org:8080/22803 Reviewed-by: Michael Smith <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- tests/common/impala_cluster.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/common/impala_cluster.py b/tests/common/impala_cluster.py index 970c1e157..dd098048a 100644 --- a/tests/common/impala_cluster.py +++ b/tests/common/impala_cluster.py @@ -246,11 +246,11 @@ class ImpalaCluster(object): for impalad in self.impalads: # lookup /varz page. webserver should up already. flags = impalad.service.get_flag_current_values() - if flags['is_coordinator'] != 'true': + if flags.get('is_coordinator', 'true') != 'true': continue - if flags['stress_catalog_init_delay_ms'] != '0': + if flags.get('stress_catalog_init_delay_ms', '0') != '0': continue - if flags['use_local_catalog'] != 'false': + if flags.get('use_local_catalog', 'false') != 'false': wait_num_table = -1 impalad.wait_for_coordinator_services(sleep_interval, check_processes_still_running, wait_num_table=wait_num_table)
