dlmarion commented on code in PR #5726:
URL: https://github.com/apache/accumulo/pull/5726#discussion_r2201935479
##########
server/compaction-coordinator/src/main/java/org/apache/accumulo/coordinator/CompactionCoordinator.java:
##########
@@ -501,6 +507,20 @@ public TNextCompactionJob getCompactionJob(TInfo tinfo,
TCredentials credentials
LOG.trace("getCompactionJob called for queue {} by compactor {}", queue,
compactorAddress);
TIME_COMPACTOR_LAST_CHECKED.put(queue, System.currentTimeMillis());
+ AtomicLong queueFailures = failingQueues.get(queue);
+ if (queueFailures != null && queueFailures.get() > 10) {
Review Comment:
I'm assuming that we would need logic to capture the time that we started
skipping compactions for this queue. After some period of time, we would want
to allow compactions to be attemtped again, and increase the time interval for
a retry on a subsequent failure.
--
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]
