srdo commented on issue #3103: STORM-3486: Bump Jersey to 2.29 for JDK 11 support URL: https://github.com/apache/storm/pull/3103#issuecomment-523508489 Okay, had to do a couple more changes. Jersey has a different set of dependencies (extra Java EE jars) when included in a Java 11 build than it does in a Java 8 build. See https://github.com/eclipse-ee4j/jersey/issues/4229. In order to get the dependency set stable between Java 8 and 11, both so the license check works and so we don't end up building Java 8 binaries that won't run on Java 11, I've added the extra Java EE jars to storm-webapp and storm-opentsdb, which is where Jersey is used. Since Eclipse took over Java EE and released the EE jars under the Jakarta namespace, we now both have those jars and the originals (e.g. jaxb-api) in the project. My understanding is that the new jars are compatible with the old ones, so I haven't bothered trying to replace them with the Jakarta jars everywhere, as they are used in a lot of places. I get the impression that Eclipse is considering renaming the packages for the Jakarta jars so EE is no longer under the `javax` namespace. See https://openliberty.io/blog/2019/05/08/jakarta-ee-naming.html. Assuming they do this, there won't be a conflict for new versions of the EE jars with the old jars, so we won't have to try to exclude the old jars from the build.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
