Make buffer size of RollingRandomAccessFileAppender configurable similar to what was done for RandomAccessFileAppender in LOG4J2-402.
I've attached a possible implementation.
Oh wow, that is fast! Would it be possible for you to submit this as a patch?
I don't know if I have permissions to commit changes but I don't think so.
You can only commit if you are a committer, which you would know.
Please attach your changes as a diff file and we'll take a look.
Here, I converted his files into a patch. Had to fix an error (probably due to an API change at some point). I could commit this as the tests all pass, but I figured I'd get some review, too.
Whoops, didn't notice he used tabs. Converted to spaces.
The unit tests should at the very least make sure that the value configured is used, perhaps by checking the ivar and the allocated buffer.
Fixed in revision 1592539.
Please verify and close.
Again, you're the reporter! Closing.
I think it was my intention that Luigi (who pointed this out to us in LOG4J2-402) would close this issue. I didn't clearly mention that though. It is fine to close it, thanks!