k.venureddy2...@gmail.com 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 <riza.sumi...@cloudera.com> Gerrit-Reviewer: Anonymous Coward <k.venureddy2...@gmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Comment-Date: Mon, 08 Sep 2025 17:03:05 +0000 Gerrit-HasComments: Yes
