[logging-log4j2] branch release-2.x updated: LOG4J2-3150: RandomAccessFileAppender uses the correct default buffer size

[ckozak]

This is an automated email from the ASF dual-hosted git repository.

ckozak pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/release-2.x by this push:
     new 5a940e0  LOG4J2-3150: RandomAccessFileAppender uses the correct 
default buffer size
5a940e0 is described below

commit 5a940e05b9cf48e1c3522be59fb876ddb38d261e
Author: Carter Kozak <cko...@apache.org>
AuthorDate: Wed Aug 25 17:19:03 2021 -0400

    LOG4J2-3150: RandomAccessFileAppender uses the correct default buffer size
---
 .../logging/log4j/core/appender/RandomAccessFileAppender.java     | 8 ++++++--
 src/changes/changes.xml                                           | 4 ++++
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/RandomAccessFileAppender.java
 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/RandomAccessFileAppender.java
index 5d0196b..2775afb 100644
--- 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/RandomAccessFileAppender.java
+++ 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/RandomAccessFileAppender.java
@@ -62,16 +62,20 @@ public final class RandomAccessFileAppender extends 
AbstractOutputStreamAppender
         @PluginBuilderAttribute("advertiseURI")
         private String advertiseURI;
 
+        public Builder() {
+            this.withBufferSize(RandomAccessFileManager.DEFAULT_BUFFER_SIZE);
+        }
+
         @Override
         public RandomAccessFileAppender build() {
             final String name = getName();
             if (name == null) {
-                LOGGER.error("No name provided for FileAppender");
+                LOGGER.error("No name provided for RandomAccessFileAppender");
                 return null;
             }
 
             if (fileName == null) {
-                LOGGER.error("No filename provided for FileAppender with name 
" + name);
+                LOGGER.error("No filename provided for 
RandomAccessFileAppender with name {}", name);
                 return null;
             }
             final Layout<? extends Serializable> layout = getOrCreateLayout();
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 57806a7..cace367 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -95,6 +95,10 @@
        based on performance improvements in modern Java releases.
       </action>
       <!-- FIXES -->
+      <action issue="LOG4J2-3150" dev="ckozak" type="fix">
+        RandomAccessFile appender uses the correct default buffer size of 256 
kB
+        rather than the default appender buffer size of 8 kB.
+      </action>
       <action issue="LOG4J2-3142" dev="ckozak" type="fix" due-to="John Meikle">
         log4j-1.2-api implements LogEventAdapter.getTimestamp() based on the 
original event timestamp
         instead of returning zero.