k.venureddy2...@gmail.com has posted comments on this change. ( http://gerrit.cloudera.org:8080/21478 )
Change subject: IMPALA-13120: Load failed table without need for manual invalidate ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/21478/4/fe/src/main/java/org/apache/impala/analysis/Analyzer.java File fe/src/main/java/org/apache/impala/analysis/Analyzer.java: http://gerrit.cloudera.org:8080/#/c/21478/4/fe/src/main/java/org/apache/impala/analysis/Analyzer.java@3703 PS4, Line 3703: Preconditions.checkState(table.isLoaded()); > Could you add an error message to checkState() as we hit this in the previo It happened due to a bug introduced in patchset-2. Actually when table is not loaded, we keep reprioritizing load for the missing tables until they are loaded. So this assert should never fail. But due to the bug in ps-2, table is not considered as missing even though it is not loaded. So the assert failed here. -- To view, visit http://gerrit.cloudera.org:8080/21478 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia882fdd865ef716351be7f1eaf203a9fb04c1c15 Gerrit-Change-Number: 21478 Gerrit-PatchSet: 4 Gerrit-Owner: Anonymous Coward <k.venureddy2...@gmail.com> Gerrit-Reviewer: Anonymous Coward <k.venureddy2...@gmail.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com> Gerrit-Comment-Date: Tue, 25 Jun 2024 15:23:14 +0000 Gerrit-HasComments: Yes
