Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/23805 )
Change subject: IMPALA-14637: COMMIT_TXN events should trigger reload for truncate ops ...................................................................... Patch Set 12: (5 comments) http://gerrit.cloudera.org:8080/#/c/23805/12//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23805/12//COMMIT_MSG@29 PS12, Line 29: accordiingly nit: typo http://gerrit.cloudera.org:8080/#/c/23805/12/fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java File fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java: http://gerrit.cloudera.org:8080/#/c/23805/12/fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java@987 PS12, Line 987: (handleIfInCatchUpMode()) Should we remove the truncate ops in an else branch? http://gerrit.cloudera.org:8080/#/c/23805/12/fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java@1221 PS12, Line 1221: tracted tracked? http://gerrit.cloudera.org:8080/#/c/23805/12/fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java@1222 PS12, Line 1222: HIVE-29677 Will our tests fail when HIVE-29677 resolves? http://gerrit.cloudera.org:8080/#/c/23805/12/fe/src/main/java/org/apache/impala/catalog/Catalog.java File fe/src/main/java/org/apache/impala/catalog/Catalog.java: http://gerrit.cloudera.org:8080/#/c/23805/12/fe/src/main/java/org/apache/impala/catalog/Catalog.java@1015 PS12, Line 1015: truncateOps_.compute Should we skip it if writeId_ <= 0? E.g. in case of Apache Hive 3? -- To view, visit http://gerrit.cloudera.org:8080/23805 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I89aac12819f08dd9ed42d5d8b21a96c04b04d75c Gerrit-Change-Number: 23805 Gerrit-PatchSet: 12 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: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Fri, 03 Jul 2026 15:33:00 +0000 Gerrit-HasComments: Yes
