malinjawi opened a new pull request, #12592: URL: https://github.com/apache/gluten/pull/12592
Fixes #12504. `getJoin()` dereferences `JniHashTableContext::vm_`, which is only set from `JNI_OnLoad`. In a process with no JVM (micro benchmark, native tests) it is null, so converting a plan with a non-empty `hashTableId` segfaults. The `try`/`catch` at the call site cannot catch a null deref. Returns a cache miss instead, so the caller builds from the build side input. Adds the missing `checkException`/`DeleteLocalRef` in `callJavaGet`, and a regression test that runs in the no-JVM configuration. Not yet compiled locally — relying on CI. Note this does not make the micro benchmark *support* BHJ; the build side is still dumped empty. Design for that is in #12504. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
