katstack commented on PR #4234: URL: https://github.com/apache/logging-log4j2/pull/4234#issuecomment-5872122595
Hi @ppkarwasz, thank you for the detailed feedback. Our use case is an air-gapped bare-metal environment where local disk capacity is constrained, while CPU headroom is available during rollover. The customer requirement is to retain application logs locally for as long as possible, so trading additional rollover CPU time for a better compression ratio is valuable. For a representative, highly repetitive application-log workload, we observed that Zstd level 9 reduced the archive to roughly 1% of the original size. This is workload-specific rather than a general Zstd result, but it illustrates why operators need to tune the CPU/compression-ratio trade-off instead of always using a fixed default. I will move `ZstdCompressAction` to `actions.internal`, then update the PR to provide consistent `compressionLevel` support for Deflate, GZip, and BZip2. I will also extract the shared compression logic into an internal base class. -- 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]
