[ https://issues.apache.org/jira/browse/HADOOP-12295?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Yi Liu updated HADOOP-12295: ---------------------------- Description: In {{NetworkTopology#InnerNode#remove}}, We can use {{childrenMap}} to get the parent node, no need to loop the {{children}} list. Another nit in current code is: {code} String parent = n.getNetworkLocation(); String currentPath = getPath(this); {code} can be in closure of {{\!isAncestor\(n\)}} was:In {{NetworkTopology#InnerNode#remove}}, We can use {{childrenMap}} to get the parent node, no need to loop the {{children}} list. > Improve NetworkTopology#InnerNode#remove logic > ---------------------------------------------- > > Key: HADOOP-12295 > URL: https://issues.apache.org/jira/browse/HADOOP-12295 > Project: Hadoop Common > Issue Type: Improvement > Reporter: Yi Liu > Assignee: Yi Liu > > In {{NetworkTopology#InnerNode#remove}}, We can use {{childrenMap}} to get > the parent node, no need to loop the {{children}} list. > Another nit in current code is: > {code} > String parent = n.getNetworkLocation(); > String currentPath = getPath(this); > {code} > can be in closure of {{\!isAncestor\(n\)}} -- This message was sent by Atlassian JIRA (v6.3.4#6332)