Csaba Ringhofer 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 3: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/22994/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/22994/3//COMMIT_MSG@28 PS3, Line 28: the Coordinator's catalog version : instead of num DB and tables cached. Good idea, it is more informative / less misleading than the db/table count. 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: e to open client ports within {num_seco > Removed wait_catalog_version. >He add the "progress log" first time at To be precise it wasn't added there, it was just moved earlier in the startup process. The patch moved wait_for_num_known_live_backends to the last step, as it waits for something that needs all coordinators to be up, so waiting for wait_for_num_known_live_backends hid the catalog related message. -- 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: 3 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: Wed, 11 Jun 2025 05:19:32 +0000 Gerrit-HasComments: Yes
