[email protected] has posted comments on this change. ( http://gerrit.cloudera.org:8080/24629 )
Change subject: IMPALA-14618: Fix EventProcessor race with catalog reset ...................................................................... Patch Set 1: (2 comments) Issue is applicable for add table and remove table. Might have to check for other operations/events too. http://gerrit.cloudera.org:8080/#/c/24629/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java: http://gerrit.cloudera.org:8080/#/c/24629/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@844 PS1, Line 844: public boolean removeTableIfNotAddedLater(long eventId, Similar issue applicable for remove table. Since issue can add table and remove table. Need to check for rename table as well. http://gerrit.cloudera.org:8080/#/c/24629/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@888 PS1, Line 888: public boolean addTableIfNotRemovedLater(long eventId, We can similar issue for add table. One scenario: Lets consider the following order of sequence - Add Table(t1), Remove table(t1), Reset. Add table is being processed in processing thread(thread switched at some point before getdb() in this method). And in reset process thread, rebuild db cache is in-progress, but invalidatedb is finished for the db that has add table(t1) in-progress. Finally, add table processing end up adding the table that is not existant. So show tables shows the non-existant table. And desc formatted throws TableLoadingException. -- To view, visit http://gerrit.cloudera.org:8080/24629 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifa0833c09cd7ec2461a0f2d1db7ab5a72ab6822b Gerrit-Change-Number: 24629 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Nandor Kollar <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]> Gerrit-Reviewer: Yida Wu <[email protected]> Gerrit-Comment-Date: Tue, 11 Aug 2026 14:02:26 +0000 Gerrit-HasComments: Yes
