[GitHub] vruusmann edited a comment on issue #23890: [SPARK-26986][ML] Add JAXB reference impl to build for Java 9+

[GitBox <gi...@apache.org>]

vruusmann edited a comment on issue #23890: [SPARK-26986][ML] Add JAXB reference impl to build for Java 9+
URL: https://github.com/apache/spark/pull/23890#issuecomment-467186211
 
 
   The JAXB reference implementation is not a monolith, but a collection of many libraries. Depending on the use case - "1) unmarshal XML only, 2) marshal XML only or 3) both marshal and unmarshal XML" - it should be possible to narrow down this collection.
   
   Apache Spark MLlib "implements" the second use case ("marshal XML only"), and there are no plans to implement the other two use cases in the foreseeable future? If so, then it should be possible to eliminate maybe two or three libraries. Can't say which off the top of my head though.
   
   Is this kind of "optimization of incoming dependencies" relevant at all? If it is, then maybe I should take some time to experiment and document this stuff (provided that there is no prior art available).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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