Hello Steve Carlin, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24652
to look at the new patch set (#4).
Change subject: IMPALA-15237: Add Calcite regression harness
......................................................................
IMPALA-15237: Add Calcite regression harness
Expose the optimized logical plan immediately before Impala
conversion to same-package tests. Preserve the production optimization
path. Capture client output labels and exact Calcite types before
conversion, then verify that conversion to ImpalaPlanRel completes.
Exercise direct Calcite parsing, analysis, logical optimization, and
Impala conversion without Frontend fallback. Use synthetic tables for a
compact supported corpus covering values, filters, aggregation, sorting,
joins, unions, and analytic functions. Characterize output schemas and
percent_rank rewriting, and explicitly classify unsupported complex
types.
Testing:
- Focused Linux CMake generation and ninja fesupport/java build
- CalciteOptimizerTest: 4 tests, 0 failures, 0 errors; BUILD SUCCESS
- git diff --check
Change-Id: I8b0a1aca63b494d9c64fa6b8e33328bedce9c6b4
Assisted-by: gpt-5.6-sol (OpenAI Codex)
---
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteOptimizer.java
A
java/calcite-planner/src/test/java/org/apache/impala/calcite/service/CalciteOptimizerTest.java
2 files changed, 301 insertions(+), 12 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/52/24652/4
--
To view, visit http://gerrit.cloudera.org:8080/24652
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8b0a1aca63b494d9c64fa6b8e33328bedce9c6b4
Gerrit-Change-Number: 24652
Gerrit-PatchSet: 4
Gerrit-Owner: Aleksandr Efimov <[email protected]>
Gerrit-Reviewer: Aleksandr Efimov <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
