vlsi commented on PR #5278: URL: https://github.com/apache/calcite/pull/5278#issuecomment-5829485451
Yes, and the PR now updates that step. The CALCITE-6616 note no longer applies: the code is compiled by JDK 25 with `--release 8`, so it can use only Java 8 APIs, and the JDK you run Gradle on does not affect the bytecode. The CI jobs that test on Java 8 then check the runtime behavior. So after this change, release managers don't really need to run on Java 8. The step still asks for JDK 8 only because I wanted a minimal change that gets us past the compiler bugs. It now also says that JDK 25 must be installed, since Gradle on Java 8 cannot download it (see [The JDK that compiles Calcite](https://github.com/apache/calcite/blob/b2d2b36e22/site/_docs/howto.md#the-jdk-that-compiles-calcite)). I can lift the JDK 8 requirement if you want. -- 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]
