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

Reply via email to