andrewlamb-est commented on PR #3542:
URL: https://github.com/apache/avro/pull/3542#issuecomment-3485708241

   > What is the minimum required version of Java for Jetty 12 ? Its 
META-INF/MANOFEST.MF file says `Build-Jdk-Spec: 22`. `javap` reports `major 
version: 61`, i.e. JDK 17.
   > 
   > Avro currently requires JDK 11+. The CI workflow is supposed to test with 
JDK 11, JDK 17 and JDK 21. I wonder how the JDK 11 build passes ?!
   
   According to the jetty documentation  Jetty 12 needs Java 17. (ref: 
https://github.com/jetty/jetty.project/issues/10485 | 
https://jetty.org/docs/jetty/12.1/index.html, etc).  
   
   I did make one change relating to the java version in the top level pom file 
configuration under enforceBytecodeVersion maxJdkVersion to 17, as the build 
failed without this change (message was something like EnforceBytecodeVersion 
failed, found bad dependency).  Perhaps this is the only place that was 
restricting the java version to ensure compatibility with Java 11?  
   
   If avro still requires JDK 11, perhaps it is not possible to update the 
jetty version at this time, even though jetty 9-to-11 are all end of life now? 
(particularly concerning in relation to any current and future CVEs relating to 
no-longer-supported dependencies)


-- 
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]

Reply via email to