[HDDS-8826] Avoid String to UUID conversion in SCMNodeManager

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Implemented
• Fix Version/s: 1.4.0
• Component/s: SCM
• Labels: pull-request-available
• Target Version/s: 1.4.0
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 12/Jun/23 19:58
• Updated: 27/Jun/23 20:00
• Resolved: 15/Jun/23 18:28

Description

SCMNodeManager#getNodeByUuid should support UUID param to avoid unnecessary conversion.

https://github.com/apache/ozone/blob/2f71e041ab54cf7acabcf75aba8bc25e3f353490/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/node/SCMNodeManager.java#L1350-L1358

  public DatanodeDetails getNodeByUuid(String uuid) {
    if (Strings.isNullOrEmpty(uuid)) {
      LOG.warn("uuid is null");
      return null;
    }
    DatanodeDetails temp = DatanodeDetails.newBuilder()
        .setUuid(UUID.fromString(uuid)).build();
    try {
      return nodeStateManager.getNode(temp);

Issue Links

• links to: GitHub Pull Request #4896