Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24652 )
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) Reviewed-on: http://gerrit.cloudera.org:8080/24652 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Steve Carlin <[email protected]> Reviewed-by: Michael Smith <[email protected]> --- 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(-) Approvals: Impala Public Jenkins: Verified Steve Carlin: Looks good to me, but someone else must approve Michael Smith: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I8b0a1aca63b494d9c64fa6b8e33328bedce9c6b4 Gerrit-Change-Number: 24652 Gerrit-PatchSet: 5 Gerrit-Owner: Aleksandr Efimov <[email protected]> Gerrit-Reviewer: Aleksandr Efimov <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
