[GitHub] [hadoop-ozone] sodonnel commented on a change in pull request #142: HDDS-2448 Delete container command should used a thread pool

[GitBox <gi...@apache.org>]

sodonnel commented on a change in pull request #142: HDDS-2448 Delete container command should used a thread pool
URL: https://github.com/apache/hadoop-ozone/pull/142#discussion_r346443129
 
 

 ##########
 File path: hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/statemachine/DatanodeStateMachine.java
 ##########
 @@ -125,7 +125,7 @@ public DatanodeStateMachine(DatanodeDetails datanodeDetails,
         .addHandler(new DeleteBlocksCommandHandler(container.getContainerSet(),
             conf))
         .addHandler(new ReplicateContainerCommandHandler(conf, supervisor))
-        .addHandler(new DeleteContainerCommandHandler())
+        .addHandler(new DeleteContainerCommandHandler(2))
 
 Review comment:
   Yes, that "2" needs to be configurable just like for the "replicate container thread pool size" in HDDS-2450. Now HDDS-2450 has been committed, I will rebase this patch and make this thread pool configurable in the same way.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org