Adds maxRandomDelay configure to Rolling Appenders#4165
Merged
Conversation
vy
added
appenders:Rolling
ramanathan1504
approved these changes
Jul 1, 2026
ramanathan1504
left a comment
ramanathan1504
left a comment
Contributor
There was a problem hiding this comment.
Thanks for putting this PR together to align with the original design @vy. My apologies for missing @ppkarwasz's feedback in #4071.
Verified and tested the updates. LGTM 👍🏾
github-project-automation
Bot
moved this from Approved
to Merged
in Log4j pull request tracker
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #4012 by reworking #4071, which superseded #4011.
Timeline
#4012, @OpenZYK asked to delay compression to mitigate I/O overload due to concurrent roll overs.#4011to fix#4012. In reviews, @ppkarwasz suggested to delay the action chain instead of creating a new action to perform the delay.#4011was abandoned and not merged.#4071, @ramanathan1504 resurrected#4011, but did not follow @ppkarwasz's suggestion.#4071is reviewed by @vy and merged.#4011to state that#4071did not follow @ppkarwasz's suggestion.#4071.