caicancai opened a new pull request, #4992: URL: https://github.com/apache/calcite/pull/4992
https://issues.apache.org/jira/browse/CALCITE-7580 This removes the Arrow adapter's dependency on Gandiva. The adapter now uses Arrow `ValueVector` reads for projection, and evaluates the current `ArrowTranslator` filter token subset in Java. This keeps the Arrow adapter correctness path in Java and avoids requiring the native Gandiva/LLVM runtime for Arrow adapter tests. Changes: * remove `arrow-gandiva` from the Arrow module and BOM/version constraints; * remove Gandiva-based project/filter enumerators; * add a Java filter enumerator for the current Arrow filter token subset; * stop skipping Arrow tests based on Gandiva native library availability. Validated locally with JDK 11: ```bash JAVA_HOME=$(/usr/libexec/java_home -v 11) ./gradlew :arrow:checkstyleMain :arrow:autostyleJavaCheck :arrow:test --tests org.apache.calcite.adapter.arrow.ArrowAdapterTest --tests org.apache.calcite.adapter.arrow.ArrowAdapterDataTypesTest --no-build-cache ``` Result: 80 tests passed. Also checked that `:arrow:dependencies --configuration runtimeClasspath` no longer includes `arrow-gandiva`. -- 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]
