dlmarion commented on a change in pull request #2274:
URL: https://github.com/apache/accumulo/pull/2274#discussion_r710990824
##########
File path:
core/src/main/java/org/apache/accumulo/core/util/threads/ThreadPools.java
##########
@@ -58,7 +82,9 @@ public static ExecutorService
createExecutorService(AccumuloConfiguration conf,
case MANAGER_RENAME_THREADS:
return createFixedThreadPool(conf.getCount(p), "bulk move", false);
case MANAGER_FATE_THREADPOOL_SIZE:
- return createFixedThreadPool(conf.getCount(p), "Repo Runner", false);
+ ThreadPoolExecutor pool = createFixedThreadPool(conf.getCount(p),
"Repo Runner", false);
+ makeResizeable(pool, conf, p);
Review comment:
I figured it would just be easier to make one method that could modify
any of the created thread pools rather than making more methods to create a
thread pool or adding more parameters to all of the existing methods.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
