[HDDS-5355] In ContainerStateMachine, share the executor threads between the containers

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Implemented
• Fix Version/s: 1.2.0
• Component/s: Ozone Datanode
• Labels: pull-request-available
• Assignee: Tsz-wo Sze
• Reporter: Tsz-wo Sze
• Created: 16/Jun/21 11:30
• Updated: 06/Oct/21 11:05
• Resolved: 22/Jun/21 09:54

Description

//ContainerStateMachine
  private ExecutorService getCommandExecutor(
      ContainerCommandRequestProto requestProto) {
    int executorId = (int)(requestProto.getContainerID() % executors.length);
    return executors[executorId];
  }

In the code above, different containers having the same remainder (mod executors.length ) will use the same executor even if some other executors are idle.  Ideally, different containers should use a different executor if there are executors available.

Issue Links

• is related to: HDDS-5831 Remove empty TaskQueue in ContainerStateMachine
• links to: GitHub Pull Request #2350