[email protected] has posted comments on this change. ( http://gerrit.cloudera.org:8080/23382 )
Change subject: IMPALA-14400: Fix deadlock in CatalogServiceCatalog.getDbProperty() ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/23382/7/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java File fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java: http://gerrit.cloudera.org:8080/#/c/23382/7/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java@2546 PS7, Line 2546: metastoreEventProcessor_.start(); If we are starting event processor here after first reset instead of starting in JniCatalog, then MetastoreEventsProcessor processEvents and updateLatestEventId threads are not scheduled till the first reset is completed. So this deadlock itself shouldn't occur because event processing is not started. right ? do we still need waitInitialResetCompletion() ? Am i missing something ? -- To view, visit http://gerrit.cloudera.org:8080/23382 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3162472ea9531add77886bf1d0d73460ff34d07a Gerrit-Change-Number: 23382 Gerrit-PatchSet: 7 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Mon, 08 Sep 2025 17:03:05 +0000 Gerrit-HasComments: Yes
