cshannon commented on code in PR #5353:
URL: https://github.com/apache/accumulo/pull/5353#discussion_r1968592900


##########
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##########
@@ -1292,6 +1293,13 @@ boolean canSuspendTablets() {
     ThreadPools.watchCriticalScheduledTask(context.getScheduledExecutor()
         .scheduleWithFixedDelay(() -> 
ScanServerMetadataEntries.clean(context), 10, 10, MINUTES));
 
+    // TODO - create new threadpool?

Review Comment:
   Yes, TGW doesn't work in this case because because the task needs to look at 
multiple tablets together and not just one tablet at a time. Also, the task 
likely doesn't need to run very often to look for mergeable ranges where as TGW 
runs frequently



-- 
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]

Reply via email to