[ https://issues.apache.org/jira/browse/HDFS-12115?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Anu Engineer updated HDFS-12115: -------------------------------- Attachment: HDFS-12115-HDFS-7240.006.patch bq. InProgressPool.java: NIT, line 203, extra space between NodeState and getNodeState Fixed. bq. MockNodeManager.java: line 161-173, it seems this can be replaced by getNodes(nodestate).size() Fixed. bq. Ozone.proto : Add a placeholder for DECOMMISSIONING state? Fixed. bq. SCMNodeManager.java : line 413-435, like you mentioned earlier, a node may have more than 1 state, e.g both HEALTHY The reason why queryNode needs to be able to read multiple states is because it is an RPC and a network call. But getNodeState is a local call and currently only use is to get the liveness of the nodes. If and when we get to automatic membership management, we can make this modification. Since no one is calling it now, I have left it as it is for now. bq. line 491: instead of creating a new list, this can be done in java8 style return currentSet.stream().collect(Collectors.toList()); I presume you are referring to StorageContainerManager.java. The reason why I am not doing it is Line: 479. Since this is an "And", if we find any null sets for any attribute, we return an empty set. With an empty ResultSet, the return is consistent, the same set object which is always non-null. > Ozone: SCM: Add queryNode RPC Call > ---------------------------------- > > Key: HDFS-12115 > URL: https://issues.apache.org/jira/browse/HDFS-12115 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: ozone > Affects Versions: HDFS-7240 > Reporter: Anu Engineer > Assignee: Anu Engineer > Fix For: HDFS-7240 > > Attachments: HDFS-12115-HDFS-7240.001.patch, > HDFS-12115-HDFS-7240.002.patch, HDFS-12115-HDFS-7240.003.patch, > HDFS-12115-HDFS-7240.004.patch, HDFS-12115-HDFS-7240.005.patch, > HDFS-12115-HDFS-7240.006.patch > > > Add queryNode RPC to Storage container location protocol. This allows > applications like SCM CLI to get the list of nodes in various states, like > Healthy, live or Dead. -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org