caicancai opened a new pull request, #5028: URL: https://github.com/apache/calcite/pull/5028
## Summary This PR implements the rollback discussed in #5025. It contains two separate revert commits, applied in dependency order: 1. Revert `a33d6a9d` / #5001 (`[CALCITE-7539] Upgrade Arrow adapter dependencies to 16.0.0`). 2. Revert `bf2b81b2` / #4992 (`[CALCITE-7580] Remove Gandiva dependency from Arrow adapter`). The Arrow upgrade is reverted first so that restoring the Gandiva-based implementation also restores the matching Arrow 15 dependencies. ## Reason The squash merge of #4992 was based on stale file contents and inadvertently reverted fixes that had already landed on `main` for CALCITE-7562, CALCITE-7567, CALCITE-7578, and CALCITE-7588. Reverting the original changes returns `main` to a known reviewed state, including those four fixes. CALCITE-7580 can then be reopened and the intended Gandiva removal can be submitted again from the latest `main` with an isolated diff. ## Validation - `./gradlew :arrow:test :testkit:compileJava` - `./gradlew :core:test --tests org.apache.calcite.rex.RexProgramTest --tests org.apache.calcite.sql.type.SqlTypeFactoryTest --tests org.apache.calcite.test.SqlToRelConverterTest` - `./gradlew :arrow:autostyleCheck :core:autostyleCheck :testkit:autostyleCheck :bom:autostyleCheck` - `git diff --check upstream/main...HEAD` Closes #5025 as the superseding cleanup approach. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
