dlmarion commented on PR #4158: URL: https://github.com/apache/accumulo/pull/4158#issuecomment-1911050880
> Given this statement is generated per-compaction, moving this entry to debug would drastically increase the amount of entries in the compactor logs. In the Compactor server process a [log entry](https://github.com/apache/accumulo/blob/2.1/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java#L693) is emitted every N seconds (and an update sent to the coordinator with the same information) so that the user can determine if progress is actually being made. I don't think a message with these details is emitted when the compaction is completed though. Making this change would mean that a log entry would show up in both the compactor and tserver logs when compactions have completed in the respective server. Additionally, if we want to allow the user to control the level of these compaction related messages, then we could put them in a class with a named logger specific to compaction information. This would allow the user to direct these specific compaction messages to a different file if they so choose. IMO, I think this might be a good change. -- 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]
