ramanathan1504 commented on PR #3926: URL: https://github.com/apache/logging-log4j2/pull/3926#issuecomment-4291181516
Hi @vy, Apologies for the long delay here. I’d like to take this forward and implement the `ListAppender` revamp based on your earlier suggestions. My plan is to: 1. Establish consistent thread-safety using method-level synchronization, removing the current mix of `volatile` and `synchronizedList`. 2. Ensure `append()` stores `event.toImmutable()` to safely support garbage-free logging. 3. Update and clarify the Javadocs (including `countDownLatch` behavior). 4. Apply general code cleanups (diamond operators, `final` fields, etc.). 5. Add functional and concurrency tests to validate correctness and thread-safety. You mentioned having a simple recipe for testing thread safety—could you please share that so I can align the test suite with your expectations? Thanks! -- 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]
