Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/17645 )
Change subject: IMPALA-5476: Fix Catalogd restart bring about metadata is out of sync ...................................................................... Patch Set 11: (4 comments) http://gerrit.cloudera.org:8080/#/c/17645/11/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/17645/11/be/src/service/impala-server.cc@2143 PS11, Line 2143: LOG(INFO) << "Catalog service ID mismatch. Current ID: " << PrintId(cur_service_id) line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/17645/11/tests/custom_cluster/test_restart_services.py File tests/custom_cluster/test_restart_services.py: http://gerrit.cloudera.org:8080/#/c/17645/11/tests/custom_cluster/test_restart_services.py@190 PS11, Line 190: flake8: W293 blank line contains whitespace http://gerrit.cloudera.org:8080/#/c/17645/11/tests/custom_cluster/test_restart_services.py@190 PS11, Line 190: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/17645/11/tests/custom_cluster/test_restart_services.py@199 PS11, Line 199: flake8: E203 whitespace before ':' -- To view, visit http://gerrit.cloudera.org:8080/17645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9fe25f5a2a42fb432e306ef08ae35750c8f3c50c Gerrit-Change-Number: 17645 Gerrit-PatchSet: 11 Gerrit-Owner: liuyao <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: liuyao <[email protected]> Gerrit-Comment-Date: Mon, 26 Jul 2021 05:31:23 +0000 Gerrit-HasComments: Yes
