K0K0V0K commented on code in PR #7928:
URL: https://github.com/apache/hadoop/pull/7928#discussion_r2322802024
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/pom.xml:
##########
@@ -208,6 +208,12 @@
<artifactId>jersey-media-json-jettison</artifactId>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.glassfish.jaxb</groupId>
+ <artifactId>jaxb-runtime</artifactId>
+ <version>${jaxb.version}</version>
+ <scope>test</scope>
Review Comment:
I tried to upload a corrected version.
With my understanding every Hadoop based Jetty server is instance of the
`HttpServer2`, what is in the `hadoop-common`, so i added the dependency there.
Some other modules what use` hadoop-common`, but not the Jetty server are
already excludes the `jetty-server`.
I extended these cases with `jaxb-runtime` exculsion. This means: not 100%
these modules dont need `jaxb-runtime`, but pretty sure not because the Jetty.
In the httpfs modul i dont know why but seems like the `jetty-server`
transitive dependency is excluded from `hadoop-common`, but there is an
explicit dependency for it also in the pom.
I run a clean install with -Pdist and i can see the jaxb-runtime jar in the
`hadoop-dist/target/hadoop-3.5.0-SNAPSHOT/share/hadoop/common/lib/jaxb-runtime-2.3.9.jar`
path
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
