wayneguow commented on code in PR #49854: URL: https://github.com/apache/spark/pull/49854#discussion_r1948060923
########## pom.xml: ########## @@ -615,16 +615,17 @@ <groupId>org.jvnet.staxex</groupId> <artifactId>stax-ex</artifactId> </exclusion> - <!-- - SPARK-27611: Exclude redundant javax.activation implementation, which - conflicts with the existing javax.activation:activation:1.1.1 dependency. - --> <exclusion> - <groupId>jakarta.activation</groupId> Review Comment: Hmm, I found that the `jakarta.xml.bind:jakarta.xml.bind-api` exclusion in `jersey-server` can be removed, but the `com.sun.activation:jakarta.activation` exclusion in `jersey-common` can still be retained, because this dependence is currently useless. -- 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