Peter Rozsa has posted comments on this change. ( http://gerrit.cloudera.org:8080/20515 )
Change subject: IMPALA-12243: Add support for DROP PARTITION for Iceberg tables ...................................................................... Patch Set 6: (8 comments) http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java File fe/src/main/java/org/apache/impala/analysis/PartitionSet.java: http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java@203 PS5, Line 203: * Rewrites SlotRefs and FunctionCallExprs as IcebergPartitionExpr. SlotRefs > line too long (92 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java@207 PS5, Line 207: * IcebergPartitionExpr where the transform is located from the function name, and > line too long (91 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java@210 PS5, Line 210: * IcebergPartitionExpr then replaces the original SlotRef/FunctionCallExpr. For Date > line too long (91 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java@319 PS5, Line 319: try { > line too long (93 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java@336 PS5, Line 336: > line too long (98 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/analysis/PartitionSet.java@342 PS5, Line 342: > line too long (98 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/catalog/IcebergTable.java File fe/src/main/java/org/apache/impala/catalog/IcebergTable.java: http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/catalog/IcebergTable.java@440 PS5, Line 440: public void loadSchemaFromIceberg() > line too long (92 > 90) Done http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/util/IcebergUtil.java File fe/src/main/java/org/apache/impala/util/IcebergUtil.java: http://gerrit.cloudera.org:8080/#/c/20515/5/fe/src/main/java/org/apache/impala/util/IcebergUtil.java@756 PS5, Line 756: TIcebergPartitionTransformType transformType, String stringValue) > line too long (103 > 90) Done -- To view, visit http://gerrit.cloudera.org:8080/20515 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2a768ba2966f570454687e02e4e6d67df46741f9 Gerrit-Change-Number: 20515 Gerrit-PatchSet: 6 Gerrit-Owner: Peter Rozsa <[email protected]> Gerrit-Reviewer: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Tamas Mate <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Mon, 06 Nov 2023 08:47:43 +0000 Gerrit-HasComments: Yes
