Re: [PR] Moved map containing long running compactions to Monitor [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner commented on code in PR #6223:
URL: https://github.com/apache/accumulo/pull/6223#discussion_r2955666534


##########
server/monitor/src/main/java/org/apache/accumulo/monitor/next/InformationFetcher.java:
##########
@@ -296,7 +297,7 @@ public void run() {
       ThreadPools.resizePool(pool, () -> Math.max(20, (futures.size() / 20)), poolName);
 
       // Fetch external compaction information from the Manager
-      futures.add(this.pool.submit(new CompactionListFetcher(summary)));
+      futures.add(this.pool.submit(new RunningCompactionFetcher(summary, pool)));

Review Comment:
   If instead of adding this single future, a future was added for each compactor to this `futures` list that seems like it would work better w/ the later code that waits on cancels futures that take too long.



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org