yuqi1129 commented on PR #10348: URL: https://github.com/apache/gravitino/pull/10348#issuecomment-4035797381
Follow-up (format-fixed): For issue #10350, JDK versions above 17 do need the same add-opens JVM options as JDK 17 (especially java.base/java.nio for Arrow MemoryUtil). So this PR change from "JVM_VERSION -eq 17" to "JVM_VERSION -ge 17" in the three startup scripts is correct. One potential gap: bin/gravitino-optimizer.sh.template still has "if [ "$JVM_VERSION" -eq 17 ]". If optimizer can load similar reflective/Arrow paths on JDK 21+, consider aligning it to "-ge 17" for consistency. -- 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]
