jojochuang edited a comment on pull request #1973: URL: https://github.com/apache/hadoop/pull/1973#issuecomment-619292730
Hm adding HADOOP-15775 did pass the build. However that's not right since we don't actually use javax.activation so we shouldn't add it as a runtime dependency. Instead I should update this patch to relocate javax.activation classes. Update. Found the source of the extra classes. They come from jackson-module-jaxb-annotations [INFO] +- com.fasterxml.jackson.module:jackson-module-jaxb-annotations:jar:2.10.3:compile [INFO] | +- jakarta.xml.bind:jakarta.xml.bind-api:jar:2.3.2:compile [INFO] | \- jakarta.activation:jakarta.activation-api:jar:1.2.1:compile https://search.maven.org/artifact/com.fasterxml.jackson.module/jackson-module-jaxb-annotations/2.10.0.pr3/jar that was added to support JDK11. Hadoop 3.2 does not support JDK11 so we can exclude it. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
