Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/24311 )
Change subject: IMPALA-14999: Calcite planner: support Iceberg tables (part 2) ...................................................................... Patch Set 12: (5 comments) http://gerrit.cloudera.org:8080/#/c/24311/12/java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/IcebergCountStarOptimizer.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/IcebergCountStarOptimizer.java: http://gerrit.cloudera.org:8080/#/c/24311/12/java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/IcebergCountStarOptimizer.java@114 PS12, Line 114: return agg; > I think we should log the exception to know why we are not doing the count- Done http://gerrit.cloudera.org:8080/#/c/24311/12/java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/IcebergCountStarOptimizer.java@208 PS12, Line 208: size > Unused variable Done http://gerrit.cloudera.org:8080/#/c/24311/12/java/calcite-planner/src/main/java/org/apache/impala/calcite/schema/CalciteIcebergTable.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/schema/CalciteIcebergTable.java: http://gerrit.cloudera.org:8080/#/c/24311/12/java/calcite-planner/src/main/java/org/apache/impala/calcite/schema/CalciteIcebergTable.java@83 PS12, Line 83: return false; > We should log the exception. Done http://gerrit.cloudera.org:8080/#/c/24311/12/testdata/workloads/functional-query/queries/QueryTest/iceberg-compound-predicate-push-down.test File testdata/workloads/functional-query/queries/QueryTest/iceberg-compound-predicate-push-down.test: http://gerrit.cloudera.org:8080/#/c/24311/12/testdata/workloads/functional-query/queries/QueryTest/iceberg-compound-predicate-push-down.test@272 PS12, Line 272: creates a different predicate > Could you please elaborate on this? Done http://gerrit.cloudera.org:8080/#/c/24311/12/testdata/workloads/functional-query/queries/QueryTest/iceberg-partitioned-insert-default.test File testdata/workloads/functional-query/queries/QueryTest/iceberg-partitioned-insert-default.test: http://gerrit.cloudera.org:8080/#/c/24311/12/testdata/workloads/functional-query/queries/QueryTest/iceberg-partitioned-insert-default.test@299 PS12, Line 299: bool_col which is not being recognized > Or is it because we don't push down bare boolean predicates? Filed IMPALA-1 This is true. However, this problem isn't just for Iceberg. I had another test that was non-Iceberg where this problem occurred. I think it was something for parquet. It's in my to-do there, and I haven't gotten around to it yet. So I'm not sure where this should be handled. If Iceberg handled it though, that would be good. -- To view, visit http://gerrit.cloudera.org:8080/24311 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1854012b1caac63ced292b338d40074db950b42d Gerrit-Change-Number: 24311 Gerrit-PatchSet: 12 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Sat, 06 Jun 2026 20:20:54 +0000 Gerrit-HasComments: Yes
