Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/24097 )
Change subject: IMPALA-14832: Make auto-incrementing Kudu columns hidden and simplify code ...................................................................... Patch Set 3: Code-Review+1 (2 comments) We should run the test suite on this. There's some Kudu test coverage for auto_incrementing_id. http://gerrit.cloudera.org:8080/#/c/24097/1/fe/src/main/java/org/apache/impala/analysis/AlterTableAlterColStmt.java File fe/src/main/java/org/apache/impala/analysis/AlterTableAlterColStmt.java: http://gerrit.cloudera.org:8080/#/c/24097/1/fe/src/main/java/org/apache/impala/analysis/AlterTableAlterColStmt.java@202 PS1, Line 202: if (column.isHidden()) { > It isn't very crucial, I just didn't want to change this error message for Ack http://gerrit.cloudera.org:8080/#/c/24097/1/fe/src/main/java/org/apache/impala/analysis/InsertStmt.java File fe/src/main/java/org/apache/impala/analysis/InsertStmt.java: http://gerrit.cloudera.org:8080/#/c/24097/1/fe/src/main/java/org/apache/impala/analysis/InsertStmt.java@986 PS1, Line 986: if (!matchFound) { > Actually we can remove this check completely as we are iterating over table Ack -- To view, visit http://gerrit.cloudera.org:8080/24097 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5da031c5dcd93ed02a6f9a21006325c2077be1e9 Gerrit-Change-Number: 24097 Gerrit-PatchSet: 3 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Fri, 13 Mar 2026 17:10:44 +0000 Gerrit-HasComments: Yes
