[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16575101#comment-16575101
]
Wei-Chiu Chuang commented on HADOOP-15317:
--
Shouldn't this be a HDFS jira? :)
> Improve
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423526#comment-16423526
]
Hudson commented on HADOOP-15317:
-
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13915 (See
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423516#comment-16423516
]
Xiao Chen commented on HADOOP-15317:
Pushed this to all branches (trunk, branch-3.[0-1], branch-2,
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423507#comment-16423507
]
Xiao Chen commented on HADOOP-15317:
Failed tests look unrelated. Committing this
> Improve
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423267#comment-16423267
]
Lei (Eddy) Xu commented on HADOOP-15317:
+1. Thanks [~xiaochen]!
> Improve NetworkTopology
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16420072#comment-16420072
]
genericqa commented on HADOOP-15317:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16419883#comment-16419883
]
Ajay Kumar commented on HADOOP-15317:
-
+1 for patch v6
> Improve NetworkTopology chooseRandom's
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16419805#comment-16419805
]
Xiao Chen commented on HADOOP-15317:
That was intentional, as that's the number of nodes under the
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418341#comment-16418341
]
Ajay Kumar commented on HADOOP-15317:
-
[~xiaochen], patch v5 looks good. One question:
* For
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16417108#comment-16417108
]
genericqa commented on HADOOP-15317:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416876#comment-16416876
]
Xiao Chen commented on HADOOP-15317:
Thanks for the review Eddy.
[^HADOOP-15317.05.patch] addressed
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416597#comment-16416597
]
genericqa commented on HADOOP-15317:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416436#comment-16416436
]
Lei (Eddy) Xu commented on HADOOP-15317:
Thanks for working on this, [~xiaochen]
A few
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416333#comment-16416333
]
Xiao Chen commented on HADOOP-15317:
Thanks for looking at the test failures, definitely related.
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16414630#comment-16414630
]
Ajay Kumar commented on HADOOP-15317:
-
[~xiaochen], thanks for updating the patch. New change to
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16414507#comment-16414507
]
Ajay Kumar commented on HADOOP-15317:
-
Hi [~xiaochen], Seems you missed adding "special case to
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16412494#comment-16412494
]
genericqa commented on HADOOP-15317:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16412427#comment-16412427
]
Xiao Chen commented on HADOOP-15317:
Thanks for the review [~ajayydv]. Comments addressed, and added
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16405479#comment-16405479
]
Ajay Kumar commented on HADOOP-15317:
-
[~xiaochen], thanks for updating the patch. I think we can
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16403275#comment-16403275
]
Xiao Chen commented on HADOOP-15317:
Discussed with [~eddyxu] about this a little bit, uploaded
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16401149#comment-16401149
]
Ajay Kumar commented on HADOOP-15317:
-
[~xiaochen], thanks for working on this.
Two comments:
*
[
https://issues.apache.org/jira/browse/HADOOP-15317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16399967#comment-16399967
]
genericqa commented on HADOOP-15317:
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
22 matches
Mail list logo
