Make buffer size of RollingRandomAccessFileAppender configurable similar to what was done for RandomAccessFileAppender in LOG4J2-402.
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!
Again, you're the reporter! Closing.
Fixed in revision 1592539.
Please verify and close.
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.
Whoops, didn't notice he used tabs. Converted to spaces.
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.
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.
I don't know if I have permissions to commit changes but I don't think so.
Oh wow, that is fast! Would it be possible for you to submit this as a patch?
I've attached a possible implementation.