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

   @ctubbsii - I don't think these log messages make sense as metrics because 
they are information about a specific transient event. The event is bound by 
the start and stop time, and you would need to add a tag to the metric with the 
external compaction id to find it. This would cause an issue because this would 
generate a ton of unique metrics.
   
   I think it's sufficient to add a different logger in the 
CompactionCoordinator for these messages, such as:
   ```
     private static final Logger LOG = 
LoggerFactory.getLogger("CompactionStatusLogger");
   ```


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