[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107363#comment-16107363
]
Arpit Agarwal commented on HADOOP-14672:
Thanks for reviewing and committing this [~djp].
>
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106836#comment-16106836
]
Hudson commented on HADOOP-14672:
-
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #12077 (See
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105904#comment-16105904
]
Junping Du commented on HADOOP-14672:
-
v5 patch LGTM too. [~arpitagarwal], please feel free to go
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105780#comment-16105780
]
Arpit Agarwal commented on HADOOP-14672:
Thank you Sean for the review.
+1 for the v5 patch.
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105727#comment-16105727
]
Sean Busbey commented on HADOOP-14672:
--
+1 (non-binding) on v5. there'll be a test added by
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105715#comment-16105715
]
Hadoop QA commented on HADOOP-14672:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105672#comment-16105672
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~djp] addressed the review comments from
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105648#comment-16105648
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey] I am leaving OfflineEditViewer
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105628#comment-16105628
]
Lei (Eddy) Xu commented on HADOOP-14672:
Hi, [~busbey]. Filed HDFS-12221 to track OEV change.
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105623#comment-16105623
]
Sean Busbey commented on HADOOP-14672:
--
Right, it used the old proprietary sun XML classes. those
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105608#comment-16105608
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey] previously it is used with default
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105596#comment-16105596
]
Sean Busbey commented on HADOOP-14672:
--
let me know when there's a jira. Seems like we ought to be
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105590#comment-16105590
]
Lei (Eddy) Xu commented on HADOOP-14672:
[~andrew.wang] it should be easy to replace xerces with
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105466#comment-16105466
]
Andrew Wang commented on HADOOP-14672:
--
[~eddyxu] how feasible do you think it is to migrate oiv
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105419#comment-16105419
]
Sean Busbey commented on HADOOP-14672:
--
okay, we need plenty of things in hadoop-hdfs, but we
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16104365#comment-16104365
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey]
[INFO] Including
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16104325#comment-16104325
]
Sean Busbey commented on HADOOP-14672:
--
{quote}
Ideally, it is true that these test jars are not
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16103825#comment-16103825
]
Sean Busbey commented on HADOOP-14672:
--
Give me tonight to figure out if I have concerns.
> Shaded
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16103814#comment-16103814
]
Junping Du commented on HADOOP-14672:
-
[~busbey], any further comments here? If not, I will go ahead
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100701#comment-16100701
]
Junping Du commented on HADOOP-14672:
-
If so, I am good with current approach.
[~busbey], do you
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100559#comment-16100559
]
Bharat Viswanadham commented on HADOOP-14672:
-
According to my understanding shading Xerces
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100538#comment-16100538
]
Junping Du commented on HADOOP-14672:
-
bq. I thought we were in agreement that they shouldn't be in
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100425#comment-16100425
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey] hadoop-hdfs project is dependent on
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100390#comment-16100390
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey] created a
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100165#comment-16100165
]
Sean Busbey commented on HADOOP-14672:
--
{quote}
Yes, they are used in the tests internally. I don't
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099532#comment-16099532
]
Hadoop QA commented on HADOOP-14672:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099286#comment-16099286
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey]
Yes, they are used in the tests
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099271#comment-16099271
]
Sean Busbey commented on HADOOP-14672:
--
I thought we were in agreement that they shouldn't be in
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099268#comment-16099268
]
Bharat Viswanadham commented on HADOOP-14672:
-
For the rest of the stuff, I am not sure, and
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099245#comment-16099245
]
Sean Busbey commented on HADOOP-14672:
--
copying over here from HADOOP-14089, I agree that we should
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099179#comment-16099179
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey]
I tried your patch. below is the
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099163#comment-16099163
]
Sean Busbey commented on HADOOP-14672:
--
[~bharatviswa] would you mind taking a look at the proposed
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099161#comment-16099161
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~djp][~busbey]
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16097019#comment-16097019
]
Hadoop QA commented on HADOOP-14672:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096989#comment-16096989
]
Bharat Viswanadham commented on HADOOP-14672:
-
Hi Sean,
It is pulled from
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096904#comment-16096904
]
Sean Busbey commented on HADOOP-14672:
--
{quote}
They are same, as packaged with JDK.
So, can we
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096900#comment-16096900
]
Hadoop QA commented on HADOOP-14672:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096898#comment-16096898
]
Bharat Viswanadham commented on HADOOP-14672:
-
[~busbey] They are same, as packaged with
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096861#comment-16096861
]
Sean Busbey commented on HADOOP-14672:
--
Those packages are also in the JDK. Is xml-API providing
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096857#comment-16096857
]
Hadoop QA commented on HADOOP-14672:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16096825#comment-16096825
]
Bharat Viswanadham commented on HADOOP-14672:
-
Hi [~djp][~busbey]
org/w3c/dom/,
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16095392#comment-16095392
]
Sean Busbey commented on HADOOP-14672:
--
if they're actually in the jdk, then we ought not relocate
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16095360#comment-16095360
]
Junping Du commented on HADOOP-14672:
-
Thanks for working on this, [~bharatviswa]!
bq. org/w3c/dom/,
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16094060#comment-16094060
]
Hadoop QA commented on HADOOP-14672:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16094034#comment-16094034
]
Bharat Viswanadham commented on HADOOP-14672:
-
Hi [~djp]
org/w3c/dom/, org/w3c/dom/,
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16093931#comment-16093931
]
Junping Du commented on HADOOP-14672:
-
bq. let's just fold this under HADOOP-14089. They have the
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16093874#comment-16093874
]
Sean Busbey commented on HADOOP-14672:
--
let's just fold this under HADOOP-14089. They have the same
[
https://issues.apache.org/jira/browse/HADOOP-14672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16093871#comment-16093871
]
Junping Du commented on HADOOP-14672:
-
I also see some mapreduce, hdfs, common classes are unshaded.
48 matches
Mail list logo