walterddr commented on pull request #7431: URL: https://github.com/apache/pinot/pull/7431#issuecomment-919569202
> Note that Java 8 compatibility does not require building with JDK8, it requires producing classfiles with version 52.0 linked against the JDK8 bootclasspath, so that the classes can be loaded on JDK8. This can be done with later versions of javac by setting the `release` flag. The advantage of using later versions of javac is that you can compile code to be packaged in Multi-Release jars, so you could, say, use foreign memory access on JDK17 but use `sun.misc.Unsafe` on JDK8. The requirement to _build with JDK8_ precludes this, or at the very least makes it much harder than it needs to be. I will create an issue to track this. totally agree. should we just do `<release>8</release>` on higher jdk versions? ( i felt like we should, since currently we only test them in CI). -- 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]
