Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23800 )

Change subject: IMPALA-14555: Add Iceberg support for SHOW PARTITIONS WHERE
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23800/2/fe/src/main/java/org/apache/impala/analysis/IcebergPartitionExpr.java
File fe/src/main/java/org/apache/impala/analysis/IcebergPartitionExpr.java:

http://gerrit.cloudera.org:8080/#/c/23800/2/fe/src/main/java/org/apache/impala/analysis/IcebergPartitionExpr.java@a61
PS2, Line 61:
You mentioned that this is an existing bug.
Can you please separate this out as its own patch and merge it first? That way, 
this bug fix is not tied with your new feature patch.



--
To view, visit http://gerrit.cloudera.org:8080/23800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c0ee4d171ae939770725d89dc504e13f82a7688
Gerrit-Change-Number: 23800
Gerrit-PatchSet: 2
Gerrit-Owner: Arnab Karmakar <[email protected]>
Gerrit-Reviewer: Arnab Karmakar <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Mon, 05 Jan 2026 21:28:21 +0000
Gerrit-HasComments: Yes

Reply via email to