Peter Rozsa has posted comments on this change. ( http://gerrit.cloudera.org:8080/24944 )
Change subject: IMPALA-15364: Reject ALTER TABLE ADD COLUMNS with NOT NULL and no default on Iceberg tables ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Csaba, if you have some time, please take a look. http://gerrit.cloudera.org:8080/#/c/24944/1/fe/src/main/java/org/apache/impala/analysis/AlterTableAddColsStmt.java File fe/src/main/java/org/apache/impala/analysis/AlterTableAddColsStmt.java: http://gerrit.cloudera.org:8080/#/c/24944/1/fe/src/main/java/org/apache/impala/analysis/AlterTableAddColsStmt.java@135 PS1, Line 135: // For Iceberg tables, only default values are supported as column options nit: indent -- To view, visit http://gerrit.cloudera.org:8080/24944 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9bf294664ef83d26ca90db28df0993805d5ce67e Gerrit-Change-Number: 24944 Gerrit-PatchSet: 1 Gerrit-Owner: Nandor Kollar <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Nandor Kollar <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Comment-Date: Tue, 29 Sep 2026 12:17:01 +0000 Gerrit-HasComments: Yes
