[HDDS-10830] Replace ConcurrentHashMap with HashMap protected by ReadWriteLock in NodeStateMap

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Done
• Fix Version/s: 2.0.0
• Component/s: SCM
• Labels: pull-request-available
• Assignee: Andrei Mikhalev
• Reporter: Andrei Mikhalev
• Created: 08/May/24 07:45
• Updated: 09/May/24 13:15
• Resolved: 09/May/24 13:15

Description

There is a class NodeStateMap with 3 fields:

1. ConcurrentHashMap<UUID, DatanodeInfo> nodeMap
2. ConcurrentHashMap<UUID, Set<ContainerID>> nodeToContainer
3. private final ReadWriteLock lock

Both read and write in NodeStateMap are protected by lock.

We can safely replace ConcurrentHashMap with pure HashMap

Issue Links

• links to: GitHub Pull Request #6654