Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22994 )
Change subject: IMPALA-14130: Wait catalog.curr-version in start-impala-cluster.py ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/22994/2/tests/common/impala_cluster.py File tests/common/impala_cluster.py: http://gerrit.cloudera.org:8080/#/c/22994/2/tests/common/impala_cluster.py@672 PS2, Line 672: catalog_version >= wait_catalog_version Csaba mention that pulling catalog metrics are informational and should not be part of waiting criteria here. He add the "progress log" first time at https://gerrit.cloudera.org/c/21656/10/tests/common/impala_cluster.py But it never meant to wait until some catalog metric in Coordinator reach some value. It just there to describe Impalad progress. Frontend (Coordinator) will not open its port until it heard first topic update from CatalogD. It is my mistake to assume that the catalog metric is important to wait. https://gerrit.cloudera.org/c/22634/20/tests/common/impala_cluster.py -- To view, visit http://gerrit.cloudera.org:8080/22994 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4a3956417ec83de4fb3fc2ef1e72eb3641099f02 Gerrit-Change-Number: 22994 Gerrit-PatchSet: 2 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Comment-Date: Tue, 10 Jun 2025 17:33:04 +0000 Gerrit-HasComments: Yes
