vy commented on PR #4011:
URL: https://github.com/apache/logging-log4j2/pull/4011#issuecomment-4830706093

   > Hi @vy @ppkarwasz, we noticed that #4071 has been merged into 2.26.0,
   
   No, **fortunately** it doesn't! :sweat_smile: It was merged to `2.x` and 
planned for `2.27.0`.
   
   > which adds compression delay support at the `Action` level 
(`GzCompressAction`/`ZipCompressAction`).
   > 
   > Our approach in this PR is different: we apply the delay at the 
`RollingFileManager` scheduling layer using `ScheduledThreadPoolExecutor`, 
which aligns with @ppkarwasz's earlier suggestion to avoid executing actions in 
parallel and prevent subtle bugs.
   > 
   > Could you let us know whether this PR is still worth pursuing, or if the 
feature is considered covered by #4071? Happy to close it if it's no longer 
needed.
   
   @OpenZYK, it is super awesome that you still follow this feature request. 
:star_struck: I will take care of submitting a new PR steering the incorrectly 
implemented #4071 in the direction @ppkarwasz suggested, and ask for your 
review.


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