vy commented on PR #3926: URL: https://github.com/apache/logging-log4j2/pull/3926#issuecomment-3337437956
@bjmi, I am really reluctant to _"Can you fix this aspect of X for my particular use case, please?"_ requests, in particular, for `ListAppender`. It has been touched by several such attempts, and now it is in a pretty inconsistent state: - JavaDoc states that _"This appender is not thread-safe."_ - `volatile` and `synchronizedList` used arbitrarily to serve for somebody's use case @bjmi, would you be interested in revamping this tool? What I have in mind is: 1. Simplify and establish complete thread-safety: remove `synchronizedList` et al. usages and mark all public methods `synchronized` 2. Improve class JavaDoc 3. Fix `countDownLatch` JavaDoc 4. Apply trivial IDE suggestions (e.g., for redundant explicit type arguments) 5. Test `ListAppender` 6. Test `ListAppender` thread safety (I can provide a simple recipe for this) -- 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]
