dajiaohuang commented on PR #4319: URL: https://github.com/apache/logging-log4j2/pull/4319#issuecomment-5893800626
Agreed: a failure in connectAndStart() must keep the pending batch because those events have not reached the database. I propose capping it at bufferSize by retrying a full batch before accepting another event; if that retry still fails, the write throws before adding the new event. This preserves the existing clear-on-commit-failure behavior. Does that overflow behavior match the intended policy? I have not pushed the local change because Maven stops at the enforcer: this checkout requires JDK [17,18), but the host has JDK 26.0.2.1. -- 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]
