[ 
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104365#comment-16104365
 ] 

Bharat Viswanadham edited comment on HADOOP-14672 at 7/28/17 2:51 AM:
----------------------------------------------------------------------

[~busbey] 
[INFO] Including org.apache.hadoop:hadoop-hdfs:jar:3.0.0-beta1-SNAPSHOT in the 
shaded jar. This includes fsimage xml tool, which is part of 
hadoop-client-minicluster jar.


The xerces:xercesImpl is added for fsimage xml tool.
Jira which added: HDFS-4629. Using com.sun.org.apache.xml.internal.serialize.* 
in XmlEditsVisitor.java is JVM vendor specific. Breaks IBM JAVA.

It is only used by fsimage xml offline tool from my understanding.



was (Author: bharatviswa):
[~busbey] 
[INFO] Including org.apache.hadoop:hadoop-hdfs:jar:3.0.0-beta1-SNAPSHOT in the 
shaded jar.
Which includes fsimage xml tool. 


The xerces:xercesImpl is added for fsimage xml tool.
Jira which added: HDFS-4629. Using com.sun.org.apache.xml.internal.serialize.* 
in XmlEditsVisitor.java is JVM vendor specific. Breaks IBM JAVA.


> Shaded Hadoop-client-minicluster include unshaded classes, like: javax, sax, 
> dom, etc.
> --------------------------------------------------------------------------------------
>
>                 Key: HADOOP-14672
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14672
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Junping Du
>            Assignee: Bharat Viswanadham
>            Priority: Blocker
>         Attachments: HADOOP-14672.02.patch, HADOOP-14672.03.patch, 
> HADOOP-14672.04.patch, HADOOP-14672.patch
>
>
> The shaded hadoop-client-minicluster shouldn't include any unshaded 
> dependencies, but we can see: javax, dom, sax, etc. are all unshaded.
> CC [~busbey]



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to