[
https://issues.apache.org/jira/browse/HADOOP-19951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18101467#comment-18101467
]
ASF GitHub Bot commented on HADOOP-19951:
-----------------------------------------
ajfabbri commented on code in PR #8654:
URL: https://github.com/apache/hadoop/pull/8654#discussion_r3706738825
##########
hadoop-client-modules/hadoop-client-minicluster/pom.xml:
##########
@@ -709,6 +709,7 @@
<exclude>ch.qos.reload4j:reload4j</exclude>
<exclude>org.eclipse.jetty.websocket:websocket-common</exclude>
<exclude>org.eclipse.jetty.websocket:websocket-api</exclude>
+ <exclude>org.eclipse.jetty:jetty-util-ajax</exclude>
Review Comment:
Similar question here.
##########
hadoop-client-modules/hadoop-client-runtime/pom.xml:
##########
@@ -168,6 +168,7 @@
so keeping it here leads to conflicts.
-->
<exclude>org.eclipse.jetty:jetty-util</exclude>
+ <exclude>org.eclipse.jetty:jetty-util-ajax</exclude>
Review Comment:
Curious why we need shading exclusion if this is no longer a dependency?
> Drop use of jetty-util-ajax and use jackson instead
> ---------------------------------------------------
>
> Key: HADOOP-19951
> URL: https://issues.apache.org/jira/browse/HADOOP-19951
> Project: Hadoop Common
> Issue Type: Task
> Reporter: PJ Fanning
> Priority: Major
> Labels: pull-request-available
>
> Hadoop uses jetty-util-ajax in a few modules to read and write JSON.
> This lib is abandoned and it is easy to switch to Jackson.
> This is a part of HADOOP-19912 that can be split out and it doesn't cause the
> same issues with worrying about jar compatibilities - it actually simplifies
> the build by removing a dependency.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
