errose28 commented on PR #8906: URL: https://github.com/apache/ozone/pull/8906#issuecomment-3185677886
@SaketaChalamchala This looks like it overlaps with #8858 which is already logging deletion information. That PR is using the existing OM system audit log, which I think we should re-use for this task as well. IMO this log is also too verbose to prevent roll-off. We just need a summary of what data was removed. Items like acks (`2025-08-06T21:57:24,557 [om1-KeyDeletingService#0] INFO OMDeletionLogger: 1 BlockGroup deletion are acked by SCM in 81 ms`) for example are not valuable long term and are only needed in the main application log. Let's define the minimal set of information about data that will be removed that we need to keep long term, put that in the existing OM system log to avoid a proliferation of log files for different purposes, and dedup any work done here and in #8858. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
