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

Tsz Wo (Nicholas), SZE commented on HADOOP-8820:
------------------------------------------------

Junping, there is no easy way to review such a big patch in HADOOP-8817.  It is 
simply too big.  Also, branch-1 is a stable branch so that we must be 
conservative.  Splitting it into 4 parts HADOOP-8820, HDFS-3941, HDFS-3942 and  
MAPREDUCE-4660 sounds really good.  Would you mind doing it?

In the meantime, I will review your balancer patch for trunk.
                
> Backport HADOOP-8469 and HADOOP-8470: add "NodeGroup" layer in new 
> NetworkTopology (also known as NetworkTopologyWithNodeGroup)
> -------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-8820
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8820
>             Project: Hadoop Common
>          Issue Type: New Feature
>          Components: net
>    Affects Versions: 1.0.0
>            Reporter: Junping Du
>            Assignee: Junping Du
>         Attachments: HADOOP-8820.b1.002.patch, HADOOP-8820.b1.003.patch, 
> HADOOP-8820.patch
>
>
> This patch backport HADOOP-8469 and HADOOP-8470 to branch-1 and includes:
> 1. Make NetworkTopology class pluggable for extension.
> 2. Implement a 4-layer NetworkTopology class (named as 
> NetworkTopologyWithNodeGroup) to use in virtualized environment (or other 
> situation with additional layer between host and rack).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to