Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/9458 )
Change subject: IMPALA-6583: Wait for non-empty catalog update at impalad startup ...................................................................... Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/9458/3/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java File fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java: http://gerrit.cloudera.org:8080/#/c/9458/3/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java@519 PS3, Line 519: public void setIsReady(boolean isReady) { > BE has a InProcessImpalaServer, which calls setIsReady() and tests things w I checked again and you are right. This is a really odd thing to do, but let's leave it for now. -- To view, visit http://gerrit.cloudera.org:8080/9458 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie5f09d03497543f283b39c6186ecfda2e0097c87 Gerrit-Change-Number: 9458 Gerrit-PatchSet: 4 Gerrit-Owner: Tianyi Wang <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Tianyi Wang <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 28 Feb 2018 19:13:31 +0000 Gerrit-HasComments: Yes
