ELY-1096: Remove unnecesary volatile modifier for currentSize variabl…

…e in Rotating File Audit log

src/main/java/org/wildfly/security/audit/FileAuditEndpoint.java

@@ -92,10 +92,26 @@ private void safeClose(Closeable c) {
         }
     }
 
+    /**
+     * Writes <code>bytes.length</code> bytes from the specified byte array
+     * to the underlying output stream managed by this class.
+     * The general contract for <code>write(bytes)</code> is that it must be
+     * invoked from a synchronization block surrounding one log message processing.
+     *
+     * @param bytes the data.
+     * @throws IOException if an I/O error occurs.
+     */
     protected void write(byte[] bytes) throws IOException {
         outputStream.write(bytes);
     }
 
+    /**
+     * The general contract for <code>preWrite(date)</code> is that any method override
+     * must ensure thread safety invoking this method from a synchronization block
+     * surrounding one log message processing.
+     *
+     * @param date
+     */
     protected void preWrite(Date date) {
         // NO-OP by default
     }

src/main/java/org/wildfly/security/audit/RotatingFileAuditEndpoint.java

@@ -50,7 +50,7 @@ public class RotatingFileAuditEndpoint extends FileAuditEndpoint {
 
     private String nextSuffix;
     private long nextRollover = Long.MAX_VALUE;
-    private volatile long currentSize = 0;
+    private long currentSize = 0;
 
     RotatingFileAuditEndpoint(Builder builder) throws IOException {
         super(builder);