From 58bc521d824b0e67c84439923427a121e09ebac6 Mon Sep 17 00:00:00 2001
From: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Date: Thu, 1 Feb 2024 16:47:03 +0530
Subject: [PATCH] fix: memory-netty module

---
 .../memory/netty/NettyAllocationManager.java     |  4 +---
 .../arrow/memory/netty/TestEndianness.java       | 16 ++++++++--------
 .../arrow/memory/netty/TestNettyAllocator.java   |  1 +
 3 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/java/memory/memory-netty/src/main/java/org/apache/arrow/memory/netty/NettyAllocationManager.java b/java/memory/memory-netty/src/main/java/org/apache/arrow/memory/netty/NettyAllocationManager.java
index 58354d0c2eebd..1e4e06df7e9ac 100644
--- a/java/memory/memory-netty/src/main/java/org/apache/arrow/memory/netty/NettyAllocationManager.java
+++ b/java/memory/memory-netty/src/main/java/org/apache/arrow/memory/netty/NettyAllocationManager.java
@@ -68,11 +68,9 @@ public ArrowBuf empty() {
   /**
    * The cut-off value for switching allocation strategies.
    */
-  private final int allocationCutOffValue;
 
   NettyAllocationManager(BufferAllocator accountingAllocator, long requestedSize, int allocationCutOffValue) {
     super(accountingAllocator);
-    this.allocationCutOffValue = allocationCutOffValue;
 
     if (requestedSize > allocationCutOffValue) {
       this.memoryChunk = null;
@@ -92,7 +90,7 @@ public ArrowBuf empty() {
   /**
    * Get the underlying memory chunk managed by this AllocationManager.
    * @return the underlying memory chunk if the request size is not greater than the
-   *   {@link NettyAllocationManager#allocationCutOffValue}, or null otherwise.
+   *          cutoff value provided in the constructor , or null otherwise.
    *
    * @deprecated this method will be removed in a future release.
    */
diff --git a/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestEndianness.java b/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestEndianness.java
index a782523cbc6d6..0c99062021f39 100644
--- a/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestEndianness.java
+++ b/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestEndianness.java
@@ -36,15 +36,15 @@ public void testNativeEndian() {
     final ByteBuf b = NettyArrowBuf.unwrapBuffer(a.buffer(4));
     b.setInt(0, 35);
     if (ByteOrder.nativeOrder() == ByteOrder.LITTLE_ENDIAN) {
-      assertEquals(b.getByte(0), 35);
-      assertEquals(b.getByte(1), 0);
-      assertEquals(b.getByte(2), 0);
-      assertEquals(b.getByte(3), 0);
+      assertEquals(35, b.getByte(0));
+      assertEquals(0, b.getByte(1));
+      assertEquals(0, b.getByte(2));
+      assertEquals(0, b.getByte(3));
     } else {
-      assertEquals(b.getByte(0), 0);
-      assertEquals(b.getByte(1), 0);
-      assertEquals(b.getByte(2), 0);
-      assertEquals(b.getByte(3), 35);
+      assertEquals(0, b.getByte(0));
+      assertEquals(0, b.getByte(1));
+      assertEquals(0, b.getByte(2));
+      assertEquals(35, b.getByte(3));
     }
     b.release();
     a.close();
diff --git a/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestNettyAllocator.java b/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestNettyAllocator.java
index c1f066014d538..4bc3b1530f906 100644
--- a/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestNettyAllocator.java
+++ b/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestNettyAllocator.java
@@ -39,6 +39,7 @@
 public class TestNettyAllocator {
 
   @Test
+  @SuppressWarnings("SynchronizeOnNonFinalField")
   public void testMemoryUsage() {
     final ListAppender<ILoggingEvent> memoryLogsAppender = new ListAppender<>();
     memoryLogsAppender.list = Collections.synchronizedList(memoryLogsAppender.list);