vruusmann commented on PR #49854:
URL: https://github.com/apache/spark/pull/49854#issuecomment-2646164469

   > Yes, Spark 4.x is targeting JDK 17.
   
   Perhaps it is then possible to be even more aggressive on some transitive 
dependency updates (than JPMML-Model 1.7.1 proposes).
   
   My goal is to maintain source compatibility with legacy platforms. One of 
new goals for the `1.7.X` development branches is to achieve 
cross-compilability with JavaScript/WASM, as implemented by the TeaVM platform.
   
   TLDR: The JPMML-Model dependency upgrade is highly unlikely to break 
anything. It's targeting JDK 11, whereas you are already targeting JDK 17. My 
unit and integration tests indicate that JPMML-Model runs fine on all JDK 11+ 
versions (specifically, 11, 17 and 21 LTS versions).


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to