[HDDS-8854] Avoid unnecessary DatanodeDetails creation for NodeStateManager lookup

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Implemented
• Fix Version/s: 1.4.0
• Component/s: SCM
• Labels: pull-request-available
• Target Version/s: 1.4.0
• Assignee: Attila Doroszlai Assign to me
• Reporter: Attila Doroszlai
• Created: 15/Jun/23 06:33
• Updated: 27/Jun/23 19:59
• Resolved: 23/Jun/23 06:47

Description

https://github.com/apache/ozone/blob/0b6a16abbb61049406c3a8b1cdbb5fe06c74bceb/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/node/NodeStateManager.java#L375-L377

  public DatanodeInfo getNode(DatanodeDetails datanodeDetails)
      throws NodeNotFoundException {
    return nodeStateMap.getNodeInfo(datanodeDetails.getUuid());

Some callers create DatanodeDetails just for the lookup. These could be avoided by an overloaded method accepting UUID directly.

Issue Links

• links to: GitHub Pull Request #4925