Hello Steve Carlin, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24667
to look at the new patch set (#3).
Change subject: WIP: IMPALA-15237: Test Substrait export
......................................................................
WIP: IMPALA-15237: Test Substrait export
Add a test-only Isthmus compatibility spike at the logical seam before
Impala conversion. Cover Values and Project, a named read with Filter
and Project, scalar arithmetic, and typed failure for unsupported
relations. Verify client-visible root names, protobuf serialization and
parsing, schemas, field references, and relation and function
inventories.
Pin a local 0.98.0 release-plus-fix snapshot for substrait-java PR #1065
and isolate the required Protobuf and SLF4J test classpath versions
within Surefire. The snapshot is intentionally unavailable from public
repositories. This WIP change must not merge until the fix ships in a
stable Substrait Java release and the dependency uses that release.
Upstream issue:
https://github.com/substrait-io/substrait-java/issues/1064
Upstream fix:
https://github.com/substrait-io/substrait-java/pull/1065
Testing:
- CalciteSubstraitCompatibilityTest: 4 tests, 0 failures, 0 errors
- Verified Isthmus and calcite-server are absent from impala-package
- git diff --check
Change-Id: I359d6d6f88df63a7b0fdf7c6d541ec884602852f
Assisted-by: gpt-5.6-sol (OpenAI Codex)
---
M java/calcite-planner/pom.xml
A
java/calcite-planner/src/test/java/org/apache/impala/calcite/service/CalciteSubstraitCompatibilityTest.java
2 files changed, 428 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/67/24667/3
--
To view, visit http://gerrit.cloudera.org:8080/24667
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I359d6d6f88df63a7b0fdf7c6d541ec884602852f
Gerrit-Change-Number: 24667
Gerrit-PatchSet: 3
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]>
