dlmarion commented on PR #5074:
URL: https://github.com/apache/accumulo/pull/5074#issuecomment-2491178320

   > > `CompactionCoordinator.registerMetrics` registers `MAJC_QUEUED` and 
`MAJC_RUNNING`. I see that QueueMetrics registers 
`COMPACTOR_JOB_PRIORITY_QUEUE_JOBS_QUEUED` with a `queue.id` tag, which could 
be summed up to replace `MAJC_QUEUED`. I don't see a direct replacement for 
`MAJC_RUNNING` though. What did you have in mind for that?
   > 
   > Ok, I see now in #5062 you said `... this information can be computed by 
summing idle metrics from compactors.` I'm wondering if that may not be 
intuitive to a user to use that for running compactions given the name.
   
   Ok, using the server idle metric to gauge whether or not the Compactor is 
running a compaction is probably not the way to go. The process has to be idle 
for so long, regulated by Property.GENERAL_IDLE_PROCESS_INTERVAL, to be 
considered idle. This is so that the metric is not constantly flapping.


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