diff --git a/docs/source/developers/java/development.rst b/docs/source/developers/java/development.rst
index 3f0ff6cdd01..9f78eccf6c5 100644
--- a/docs/source/developers/java/development.rst
+++ b/docs/source/developers/java/development.rst
@@ -137,3 +137,19 @@ This applies the style to all pom.xml files under the current directory or from
 .. _conbench: https://github.com/conbench/conbench
 .. _checkstyle: https://github.com/apache/arrow/blob/main/java/dev/checkstyle/checkstyle.xml
 .. _Apache Maven pom.xml guidelines: https://maven.apache.org/developers/conventions/code.html#pom-code-convention
+
+
+Build Caching
+=============
+
+Build caching is done through Develocity (formerly Maven Enterprise).  To force
+a build without the cache, run::
+
+    mvn clean install -Ddevelocity.cache.local.enabled=false -Ddevelocity.cache.remote.enabled=false
+
+This can be useful to make sure you see all warnings from ErrorProne, for example.
+
+ErrorProne
+==========
+
+ErrorProne should be disabled for generated code.
diff --git a/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowUtils.java b/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowUtils.java
index 1f5ad9e7689..c949f4b1ec5 100644
--- a/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowUtils.java
+++ b/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowUtils.java
@@ -27,6 +27,7 @@
 import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import java.util.Set;
 import java.util.stream.Collectors;
@@ -95,7 +96,6 @@
 import org.apache.avro.LogicalType;
 import org.apache.avro.LogicalTypes;
 import org.apache.avro.Schema;
-import org.apache.avro.Schema.Type;
 import org.apache.avro.io.Decoder;
 
 /**
@@ -159,7 +159,7 @@ private static Consumer createConsumer(
 
     final BufferAllocator allocator = config.getAllocator();
 
-    final Type type = schema.getType();
+    final Schema.Type type = schema.getType();
     final LogicalType logicalType = schema.getLogicalType();
 
     final ArrowType arrowType;
@@ -215,7 +215,7 @@ private static Consumer createConsumer(
           vector = createVector(consumerVector, fieldType, name, allocator);
           consumer = new AvroTimeMillisConsumer((TimeMilliVector) vector);
         } else {
-          arrowType = new ArrowType.Int(32, /*signed=*/true);
+          arrowType = new ArrowType.Int(32, /*isSigned=*/true);
           fieldType = new FieldType(nullable, arrowType, /*dictionary=*/null, getMetaData(schema));
           vector = createVector(consumerVector, fieldType, name, allocator);
           consumer = new AvroIntConsumer((IntVector) vector);
@@ -244,7 +244,7 @@ private static Consumer createConsumer(
           vector = createVector(consumerVector, fieldType, name, allocator);
           consumer = new AvroTimestampMicrosConsumer((TimeStampMicroVector) vector);
         } else {
-          arrowType = new ArrowType.Int(64, /*signed=*/true);
+          arrowType = new ArrowType.Int(64, /*isSigned=*/true);
           fieldType = new FieldType(nullable, arrowType, /*dictionary=*/null, getMetaData(schema));
           vector = createVector(consumerVector, fieldType, name, allocator);
           consumer = new AvroLongConsumer((BigIntVector) vector);
@@ -278,7 +278,7 @@ private static Consumer createConsumer(
       case NULL:
         arrowType = new ArrowType.Null();
         fieldType = new FieldType(nullable, arrowType, /*dictionary=*/null, getMetaData(schema));
-        vector = fieldType.createNewSingleVector(name, allocator, /*schemaCallback=*/null);
+        vector = fieldType.createNewSingleVector(name, allocator, /*schemaCallBack=*/null);
         consumer = new AvroNullConsumer((NullVector) vector);
         break;
       default:
@@ -305,7 +305,7 @@ private static ArrowType createDecimalArrowType(LogicalTypes.Decimal logicalType
   private static Consumer createSkipConsumer(Schema schema) {
 
     SkipFunction skipFunction;
-    Type type = schema.getType();
+    Schema.Type type = schema.getType();
 
     switch (type) {
       case UNION:
@@ -391,7 +391,7 @@ static CompositeAvroConsumer createCompositeConsumer(
     final Set<String> skipFieldNames = config.getSkipFieldNames();
 
     Schema.Type type = schema.getType();
-    if (type == Type.RECORD) {
+    if (type == Schema.Type.RECORD) {
       for (Schema.Field field : schema.getFields()) {
         if (skipFieldNames.contains(field.name())) {
           consumers.add(createSkipConsumer(field.schema()));
@@ -416,7 +416,7 @@ private static FieldVector createVector(FieldVector consumerVector, FieldType fi
 
   private static String getDefaultFieldName(ArrowType type) {
     Types.MinorType minorType = Types.getMinorTypeForArrowType(type);
-    return minorType.name().toLowerCase();
+    return minorType.name().toLowerCase(Locale.ROOT);
   }
 
   private static Field avroSchemaToField(Schema schema, String name, AvroToArrowConfig config) {
@@ -429,7 +429,7 @@ private static Field avroSchemaToField(
       AvroToArrowConfig config,
       Map<String, String> externalProps) {
 
-    final Type type = schema.getType();
+    final Schema.Type type = schema.getType();
     final LogicalType logicalType = schema.getLogicalType();
     final List<Field> children = new ArrayList<>();
     final FieldType fieldType;
@@ -457,7 +457,7 @@ private static Field avroSchemaToField(
         FieldType structFieldType = new FieldType(false, new ArrowType.Struct(), /*dictionary=*/null);
         Field structField = new Field("internal", structFieldType, Arrays.asList(keyField, valueField));
         children.add(structField);
-        fieldType = createFieldType(new ArrowType.Map(/*keySorted=*/false), schema, externalProps);
+        fieldType = createFieldType(new ArrowType.Map(/*keysSorted=*/false), schema, externalProps);
         break;
       case RECORD:
         final Set<String> skipFieldNames = config.getSkipFieldNames();
@@ -509,7 +509,7 @@ private static Field avroSchemaToField(
         } else if (logicalType instanceof LogicalTypes.TimeMillis) {
           intArrowType = new ArrowType.Time(TimeUnit.MILLISECOND, 32);
         } else {
-          intArrowType = new ArrowType.Int(32, /*signed=*/true);
+          intArrowType = new ArrowType.Int(32, /*isSigned=*/true);
         }
         fieldType = createFieldType(intArrowType, schema, externalProps);
         break;
@@ -525,7 +525,7 @@ private static Field avroSchemaToField(
         } else if (logicalType instanceof LogicalTypes.TimestampMicros) {
           longArrowType = new ArrowType.Timestamp(TimeUnit.MICROSECOND, null);
         } else {
-          longArrowType = new ArrowType.Int(64, /*signed=*/true);
+          longArrowType = new ArrowType.Int(64, /*isSigned=*/true);
         }
         fieldType = createFieldType(longArrowType, schema, externalProps);
         break;
@@ -668,7 +668,7 @@ private static Consumer createUnionConsumer(Schema schema, String name, AvroToAr
       FieldVector consumerVector) {
     final int size = schema.getTypes().size();
 
-    final boolean nullable = schema.getTypes().stream().anyMatch(t -> t.getType() == Type.NULL);
+    final boolean nullable = schema.getTypes().stream().anyMatch(t -> t.getType() == Schema.Type.NULL);
 
     UnionVector unionVector;
     if (consumerVector == null) {
@@ -709,7 +709,7 @@ static VectorSchemaRoot avroToArrowVectors(
     final Set<String> skipFieldNames = config.getSkipFieldNames();
 
     Schema.Type type = schema.getType();
-    if (type == Type.RECORD) {
+    if (type == Schema.Type.RECORD) {
       for (Schema.Field field : schema.getFields()) {
         if (skipFieldNames.contains(field.name())) {
           consumers.add(createSkipConsumer(field.schema()));
diff --git a/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowVectorIterator.java b/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowVectorIterator.java
index 4a439ade811..9a0cfd97a49 100644
--- a/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowVectorIterator.java
+++ b/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/AvroToArrowVectorIterator.java
@@ -162,6 +162,7 @@ public boolean hasNext() {
   /**
    * Gets the next vector. The user is responsible for freeing its resources.
    */
+  @Override
   public VectorSchemaRoot next() {
     Preconditions.checkArgument(hasNext());
     VectorSchemaRoot returned = nextBatch;
@@ -177,6 +178,7 @@ public VectorSchemaRoot next() {
   /**
    * Clean up resources.
    */
+  @Override
   public void close() {
     if (nextBatch != null) {
       nextBatch.close();
diff --git a/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/consumers/Consumer.java b/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/consumers/Consumer.java
index c2ae1ce77b2..8eaaf74cff6 100644
--- a/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/consumers/Consumer.java
+++ b/java/adapter/avro/src/main/java/org/apache/arrow/adapter/avro/consumers/Consumer.java
@@ -53,6 +53,7 @@ public interface Consumer<T extends FieldVector> extends AutoCloseable {
   /**
    * Close this consumer when occurs exception to avoid potential leak.
    */
+  @Override
   void close() throws Exception;
 
   /**
diff --git a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroSkipFieldTest.java b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroSkipFieldTest.java
index a37eca6514e..54fa26afe3f 100644
--- a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroSkipFieldTest.java
+++ b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroSkipFieldTest.java
@@ -20,6 +20,7 @@
 import static org.junit.Assert.assertEquals;
 
 import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
@@ -223,7 +224,7 @@ public void testSkipStringField() throws Exception {
     ArrayList<GenericRecord> expectedData = new ArrayList<>();
 
     for (int i = 0; i < 5; i++) {
-      final byte[] testBytes = ("test" + i).getBytes();
+      final byte[] testBytes = ("test" + i).getBytes(StandardCharsets.UTF_8);
       GenericRecord record = new GenericData.Record(schema);
       GenericData.Fixed fixed = new GenericData.Fixed(schema.getField("f0").schema());
       fixed.bytes(testBytes);
@@ -257,7 +258,7 @@ public void testSkipBytesField() throws Exception {
     ArrayList<GenericRecord> expectedData = new ArrayList<>();
 
     for (int i = 0; i < 5; i++) {
-      final byte[] testBytes = ("test" + i).getBytes();
+      final byte[] testBytes = ("test" + i).getBytes(StandardCharsets.UTF_8);
       GenericRecord record = new GenericData.Record(schema);
       GenericData.Fixed fixed = new GenericData.Fixed(schema.getField("f0").schema());
       fixed.bytes(testBytes);
@@ -291,7 +292,7 @@ public void testSkipFixedField() throws Exception {
     ArrayList<GenericRecord> expectedData = new ArrayList<>();
 
     for (int i = 0; i < 5; i++) {
-      final byte[] testBytes = ("test" + i).getBytes();
+      final byte[] testBytes = ("test" + i).getBytes(StandardCharsets.UTF_8);
       GenericRecord record = new GenericData.Record(schema);
       GenericData.Fixed fixed = new GenericData.Fixed(schema.getField("f0").schema());
       fixed.bytes(testBytes);
@@ -325,7 +326,7 @@ public void testSkipEnumField() throws Exception {
     ArrayList<GenericRecord> expectedData = new ArrayList<>();
 
     for (int i = 0; i < 5; i++) {
-      final byte[] testBytes = ("test" + i).getBytes();
+      final byte[] testBytes = ("test" + i).getBytes(StandardCharsets.UTF_8);
       GenericRecord record = new GenericData.Record(schema);
       GenericData.Fixed fixed = new GenericData.Fixed(schema.getField("f0").schema());
       fixed.bytes(testBytes);
diff --git a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroTestBase.java b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroTestBase.java
index 60a3a285db3..a91bba7b84f 100644
--- a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroTestBase.java
+++ b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroTestBase.java
@@ -145,7 +145,7 @@ protected void checkPrimitiveResult(List data, FieldVector vector) {
     }
   }
 
-  protected void checkRecordResult(Schema schema, ArrayList<GenericRecord> data, VectorSchemaRoot root) {
+  protected void checkRecordResult(Schema schema, List<GenericRecord> data, VectorSchemaRoot root) {
     assertEquals(data.size(), root.getRowCount());
     assertEquals(schema.getFields().size(), root.getFieldVectors().size());
 
@@ -194,7 +194,7 @@ protected void checkArrayResult(List<List<?>> expected, List<ListVector> vectors
     }
   }
 
-  protected void checkRecordResult(Schema schema, ArrayList<GenericRecord> data, List<VectorSchemaRoot> roots) {
+  protected void checkRecordResult(Schema schema, List<GenericRecord> data, List<VectorSchemaRoot> roots) {
     roots.forEach(root -> {
       assertEquals(schema.getFields().size(), root.getFieldVectors().size());
     });
diff --git a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowIteratorTest.java b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowIteratorTest.java
index 02f7a373373..7f2edb08fda 100644
--- a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowIteratorTest.java
+++ b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowIteratorTest.java
@@ -181,13 +181,13 @@ public void runLargeNumberOfRows() throws Exception {
       }
     }
 
-    assertEquals(x, targetRows);
+    assertEquals(targetRows, x);
   }
 
   /**
    * Fake avro decoder to test large data.
    */
-  private class FakeDecoder extends Decoder {
+  private static class FakeDecoder extends Decoder {
 
     private int numRows;
 
diff --git a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowTest.java b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowTest.java
index 1c642041917..26f72173b6b 100644
--- a/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowTest.java
+++ b/java/adapter/avro/src/test/java/org/apache/arrow/adapter/avro/AvroToArrowTest.java
@@ -87,10 +87,8 @@ public void testFixedAttributes() throws Exception {
     Schema schema = getSchema("attrs/test_fixed_attr.avsc");
 
     List<GenericData.Fixed> data = new ArrayList<>();
-    List<byte[]> expected = new ArrayList<>();
     for (int i = 0; i < 5; i++) {
       byte[] value = ("value" + i).getBytes(StandardCharsets.UTF_8);
-      expected.add(value);
       GenericData.Fixed fixed = new GenericData.Fixed(schema);
       fixed.bytes(value);
       data.add(fixed);
diff --git a/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/BinaryConsumer.java b/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/BinaryConsumer.java
index 8c5f61169d4..538d161f9e9 100644
--- a/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/BinaryConsumer.java
+++ b/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/BinaryConsumer.java
@@ -74,7 +74,7 @@ public void consume(InputStream is) throws IOException {
         vector.getDataBuffer().setBytes(startOffset + dataLength, reuseBytes, 0, read);
         dataLength += read;
       }
-      offsetBuffer.setInt((currentIndex + 1) * VarBinaryVector.OFFSET_WIDTH, startOffset + dataLength);
+      offsetBuffer.setInt((currentIndex + 1) * ((long) VarBinaryVector.OFFSET_WIDTH), startOffset + dataLength);
       BitVectorHelper.setBit(vector.getValidityBuffer(), currentIndex);
       vector.setLastSet(currentIndex);
     }
diff --git a/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/ClobConsumer.java b/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/ClobConsumer.java
index a52d9b73b4d..3ed0c2d3cbb 100644
--- a/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/ClobConsumer.java
+++ b/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/ClobConsumer.java
@@ -88,7 +88,7 @@ public void consume(ResultSet resultSet) throws SQLException {
 
           ArrowBuf dataBuffer = vector.getDataBuffer();
           ArrowBuf offsetBuffer = vector.getOffsetBuffer();
-          int startIndex = offsetBuffer.getInt(currentIndex * 4);
+          int startIndex = offsetBuffer.getInt(currentIndex * 4L);
           while (read <= length) {
             String str = clob.getSubString(read, readSize);
             byte[] bytes = str.getBytes(StandardCharsets.UTF_8);
@@ -106,7 +106,7 @@ public void consume(ResultSet resultSet) throws SQLException {
             totalBytes += bytes.length;
             read += readSize;
           }
-          offsetBuffer.setInt((currentIndex + 1) * 4, startIndex + totalBytes);
+          offsetBuffer.setInt((currentIndex + 1) * 4L, startIndex + totalBytes);
           BitVectorHelper.setBit(vector.getValidityBuffer(), currentIndex);
           vector.setLastSet(currentIndex);
         }
@@ -139,7 +139,7 @@ public void consume(ResultSet resultSet) throws SQLException {
 
         ArrowBuf dataBuffer = vector.getDataBuffer();
         ArrowBuf offsetBuffer = vector.getOffsetBuffer();
-        int startIndex = offsetBuffer.getInt(currentIndex * 4);
+        int startIndex = offsetBuffer.getInt(currentIndex * 4L);
         while (read <= length) {
           String str = clob.getSubString(read, readSize);
           byte[] bytes = str.getBytes(StandardCharsets.UTF_8);
@@ -157,7 +157,7 @@ public void consume(ResultSet resultSet) throws SQLException {
           totalBytes += bytes.length;
           read += readSize;
         }
-        offsetBuffer.setInt((currentIndex + 1) * 4, startIndex + totalBytes);
+        offsetBuffer.setInt((currentIndex + 1) * 4L, startIndex + totalBytes);
         BitVectorHelper.setBit(vector.getValidityBuffer(), currentIndex);
         vector.setLastSet(currentIndex);
       }
diff --git a/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/JdbcConsumer.java b/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/JdbcConsumer.java
index 480dfe3a1c5..7c867c7ad64 100644
--- a/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/JdbcConsumer.java
+++ b/java/adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/JdbcConsumer.java
@@ -37,6 +37,7 @@ public interface JdbcConsumer<T extends ValueVector> extends AutoCloseable {
   /**
    * Close this consumer, do some clean work such as clear reuse ArrowBuf.
    */
+  @Override
   void close() throws Exception;
 
   /**
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcParameterBinderTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcParameterBinderTest.java
index 15b9ab03861..a94f0aa454f 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcParameterBinderTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcParameterBinderTest.java
@@ -109,8 +109,8 @@ void bindOrder() throws SQLException {
          final VectorSchemaRoot root = VectorSchemaRoot.create(schema, allocator)) {
       final JdbcParameterBinder binder =
           JdbcParameterBinder.builder(statement, root)
-              .bind(/*paramIndex=*/ 1, /*colIndex=*/ 2)
-              .bind(/*paramIndex=*/ 2, /*colIndex=*/ 0)
+              .bind(/*parameterIndex=*/ 1, /*columnIndex=*/ 2)
+              .bind(/*parameterIndex=*/ 2, /*columnIndex=*/ 0)
               .build();
       assertThat(binder.next()).isFalse();
 
@@ -169,7 +169,7 @@ void customBinder() throws SQLException {
       final JdbcParameterBinder binder =
           JdbcParameterBinder.builder(statement, root)
               .bind(
-                  /*paramIndex=*/ 1,
+                  /*parameterIndex=*/ 1,
                   new ColumnBinder() {
                     private final IntVector vector = (IntVector) root.getVector(0);
                     @Override
@@ -275,11 +275,11 @@ void time32() throws SQLException {
   @Test
   void time64() throws SQLException {
     testSimpleType(new ArrowType.Time(TimeUnit.MICROSECOND, 64), Types.TIME,
-        (valueVectors, index, value) -> valueVectors.setSafe(index, (int) (value.getTime() * 1_000)),
+        (valueVectors, index, value) -> valueVectors.setSafe(index, (value.getTime() * 1_000)),
         TimeMicroVector::setNull,
         Arrays.asList(new Time(-128_000), new Time(104_000), new Time(-42_000)));
     testSimpleType(new ArrowType.Time(TimeUnit.NANOSECOND, 64), Types.TIME,
-        (valueVectors, index, value) -> valueVectors.setSafe(index, (int) (value.getTime() * 1_000_000)),
+        (valueVectors, index, value) -> valueVectors.setSafe(index, (value.getTime() * 1_000_000)),
         TimeNanoVector::setNull,
         Arrays.asList(new Time(-128), new Time(104), new Time(-42)));
   }
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowCommentMetadataTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowCommentMetadataTest.java
index 07cab0d829f..4ee65944c3a 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowCommentMetadataTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowCommentMetadataTest.java
@@ -19,9 +19,6 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-import java.nio.charset.StandardCharsets;
-import java.nio.file.Files;
-import java.nio.file.Paths;
 import java.sql.Connection;
 import java.sql.DatabaseMetaData;
 import java.sql.DriverManager;
@@ -34,7 +31,6 @@
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Map;
-import java.util.Objects;
 import java.util.Set;
 
 import org.apache.arrow.memory.RootAllocator;
@@ -228,9 +224,4 @@ private String getColumnComment(DatabaseMetaData metaData, String tableName, Str
     }
     return null;
   }
-
-  private String getExpectedSchema(String expectedResource) throws java.io.IOException, java.net.URISyntaxException {
-    return new String(Files.readAllBytes(Paths.get(Objects.requireNonNull(
-            JdbcToArrowCommentMetadataTest.class.getResource(expectedResource)).toURI())), StandardCharsets.UTF_8);
-  }
 }
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowConfigTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowConfigTest.java
index 68a681b052c..d4fb7c32997 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowConfigTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowConfigTest.java
@@ -89,8 +89,8 @@ public void testConfig() {
     JdbcToArrowConfigBuilder builder = new JdbcToArrowConfigBuilder(allocator, calendar);
     JdbcToArrowConfig config = builder.build();
 
-    assertTrue(allocator == config.getAllocator());
-    assertTrue(calendar == config.getCalendar());
+    assertEquals(allocator, config.getAllocator());
+    assertEquals(calendar, config.getCalendar());
 
     Calendar newCalendar = Calendar.getInstance();
     BufferAllocator newAllocator = new RootAllocator(Integer.SIZE);
@@ -98,8 +98,8 @@ public void testConfig() {
     builder.setAllocator(newAllocator).setCalendar(newCalendar);
     config = builder.build();
 
-    assertTrue(newAllocator == config.getAllocator());
-    assertTrue(newCalendar == config.getCalendar());
+    assertEquals(newAllocator, config.getAllocator());
+    assertEquals(newCalendar, config.getCalendar());
   }
 
   @Test
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowTestHelper.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowTestHelper.java
index 91f2f465dd9..7dd881b3f7c 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowTestHelper.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/JdbcToArrowTestHelper.java
@@ -24,6 +24,7 @@
 
 import java.math.BigDecimal;
 import java.nio.charset.Charset;
+import java.nio.charset.StandardCharsets;
 import java.sql.ResultSetMetaData;
 import java.sql.SQLException;
 import java.util.AbstractMap;
@@ -399,7 +400,7 @@ public static byte[][] getCharArray(String[] values, String dataType) {
     byte[][] valueArr = new byte[dataArr.length][];
     int i = 0;
     for (String data : dataArr) {
-      valueArr[i++] = "null".equals(data.trim()) ? null : data.trim().getBytes();
+      valueArr[i++] = "null".equals(data.trim()) ? null : data.trim().getBytes(StandardCharsets.UTF_8);
     }
     return valueArr;
   }
@@ -419,11 +420,12 @@ public static byte[][] getBinaryValues(String[] values, String dataType) {
     byte[][] valueArr = new byte[dataArr.length][];
     int i = 0;
     for (String data : dataArr) {
-      valueArr[i++] = "null".equals(data.trim()) ? null : data.trim().getBytes();
+      valueArr[i++] = "null".equals(data.trim()) ? null : data.trim().getBytes(StandardCharsets.UTF_8);
     }
     return valueArr;
   }
 
+  @SuppressWarnings("StringSplitter")
   public static String[] getValues(String[] values, String dataType) {
     String value = "";
     for (String val : values) {
@@ -440,6 +442,7 @@ public static Integer[][] getListValues(String[] values, String dataType) {
     return getListValues(dataArr);
   }
 
+  @SuppressWarnings("StringSplitter")
   public static Integer[][] getListValues(String[] dataArr) {
     Integer[][] valueArr = new Integer[dataArr.length][];
     int i = 0;
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/ResultSetUtility.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/ResultSetUtility.java
index ccc7681c5bc..b05a59a9a04 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/ResultSetUtility.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/ResultSetUtility.java
@@ -66,17 +66,17 @@ public static MockResultSet generateBasicResultSet(int rows) throws SQLException
   }
 
   public static class MockResultSet extends ThrowingResultSet {
-    private final ArrayList<MockRow> rows;
+    private final List<MockRow> rows;
     private int index = 0;
     private boolean isClosed = false;
     private ResultSetMetaData metadata;
     private boolean wasNull;
 
-    public MockResultSet(ArrayList<MockRow> rows) throws SQLException {
+    public MockResultSet(List<MockRow> rows) throws SQLException {
       this(rows, MockResultSetMetaData.fromRows(rows));
     }
 
-    public MockResultSet(ArrayList<MockRow> rows, ResultSetMetaData metadata) {
+    public MockResultSet(List<MockRow> rows, ResultSetMetaData metadata) {
       this.rows = rows;
       this.metadata = metadata;
       this.wasNull = false;
@@ -252,8 +252,8 @@ public Builder addDataElement(Object val, int sqlType) {
         return this.addDataElement(new MockDataElement(val, sqlType));
       }
 
-      public Builder setMetaData(ResultSetMetaData metaData) {
-        this.metadata = metaData;
+      public Builder setMetaData(ResultSetMetaData metadata) {
+        this.metadata = metadata;
         return this;
       }
 
@@ -318,7 +318,7 @@ public String getColumnTypeName(int column) throws SQLException {
       return columns.get(column - 1).getTypeName();
     }
 
-    public static MockResultSetMetaData fromRows(ArrayList<MockRow> rows) throws SQLException {
+    public static MockResultSetMetaData fromRows(List<MockRow> rows) throws SQLException {
       // Note: This attempts to dynamically construct ResultSetMetaData from the first row in a given result set.
       // If there are now rows, or the result set contains no columns, this cannot be dynamically generated and
       // an exception will be thrown.
@@ -338,7 +338,6 @@ public static MockResultSetMetaData fromRows(ArrayList<MockRow> rows) throws SQL
     }
 
     public static class MockColumnMetaData {
-      private int index;
       private int sqlType;
       private int precision;
       private int scale;
@@ -385,7 +384,6 @@ private int getDisplaySize() {
 
       public static MockColumnMetaData fromDataElement(MockDataElement element, int i) throws SQLException {
         return MockColumnMetaData.builder()
-                .index(i)
                 .sqlType(element.getSqlType())
                 .precision(element.getPrecision())
                 .scale(element.getScale())
@@ -403,11 +401,6 @@ public static Builder builder() {
       public static class Builder {
         private MockColumnMetaData columnMetaData = new MockColumnMetaData();
 
-        public Builder index(int index) {
-          this.columnMetaData.index = index;
-          return this;
-        }
-
         public Builder label(String label) {
           this.columnMetaData.label = label;
           return this;
@@ -453,9 +446,9 @@ public MockColumnMetaData build() {
   }
 
   public static class MockRow {
-    private final ArrayList<MockDataElement> dataElements;
+    private final List<MockDataElement> dataElements;
 
-    public MockRow(ArrayList<MockDataElement> elements) {
+    public MockRow(List<MockDataElement> elements) {
       this.dataElements = elements;
     }
 
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/UnreliableMetaDataTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/UnreliableMetaDataTest.java
index 3eb886faabc..053604073fd 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/UnreliableMetaDataTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/UnreliableMetaDataTest.java
@@ -171,7 +171,6 @@ public void testIncorrectNullability() throws Exception {
     // ARROW-17005: ResultSetMetaData may indicate a field is non-nullable even when there are nulls
     ResultSetUtility.MockResultSetMetaData.MockColumnMetaData columnMetaData =
         ResultSetUtility.MockResultSetMetaData.MockColumnMetaData.builder()
-            .index(1)
             .sqlType(Types.INTEGER)
             .nullable(ResultSetMetaData.columnNoNulls)
             .build();
@@ -257,7 +256,6 @@ public void testIncorrectNullability() throws Exception {
   private ResultSet buildIncorrectPrecisionAndScaleMetaDataResultSet() throws SQLException {
     ResultSetUtility.MockResultSetMetaData.MockColumnMetaData columnMetaData =
         ResultSetUtility.MockResultSetMetaData.MockColumnMetaData.builder()
-            .index(1)
             .sqlType(Types.DECIMAL)
             .precision(0)
             .scale(0)
@@ -277,7 +275,6 @@ private ResultSet buildIncorrectPrecisionAndScaleMetaDataResultSet() throws SQLE
   private ResultSet buildVaryingPrecisionAndScaleResultSet() throws SQLException {
     ResultSetUtility.MockResultSetMetaData.MockColumnMetaData columnMetaData =
         ResultSetUtility.MockResultSetMetaData.MockColumnMetaData.builder()
-            .index(1)
             .sqlType(Types.DECIMAL)
             .precision(0)
             .scale(0)
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcAliasToArrowTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcAliasToArrowTest.java
index d9acfe88f4f..d32c2bbab91 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcAliasToArrowTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcAliasToArrowTest.java
@@ -111,8 +111,8 @@ public void testJdbcAliasToArrow() throws Exception {
       assertEquals(rowCount, vector.getRowCount());
       Schema vectorSchema = vector.getSchema();
       List<Field> vectorFields = vectorSchema.getFields();
-      assertEquals(vectorFields.get(0).getName(), COLUMN_A);
-      assertEquals(vectorFields.get(1).getName(), COLUMN_B);
+      assertEquals(COLUMN_A, vectorFields.get(0).getName());
+      assertEquals(COLUMN_B, vectorFields.get(1).getName());
     }
   }
 
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowArrayTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowArrayTest.java
index 377e332b43a..eabbdc5a25e 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowArrayTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowArrayTest.java
@@ -22,6 +22,7 @@
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 
+import java.nio.charset.StandardCharsets;
 import java.sql.Array;
 import java.sql.Connection;
 import java.sql.DriverManager;
@@ -284,7 +285,7 @@ private void assertStringVectorEquals(ListVector listVector, int rowCount, Strin
       assertEquals(1, listVector.isSet(row));
       assertEquals(expectedValues[row].length, offset - prevOffset);
       for (int i = prevOffset; i < offset; ++i) {
-        assertArrayEquals(expectedValues[row][i - prevOffset].getBytes(), vector.get(i));
+        assertArrayEquals(expectedValues[row][i - prevOffset].getBytes(StandardCharsets.UTF_8), vector.get(i));
       }
 
       prevOffset = offset;
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowCharSetTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowCharSetTest.java
index 422b55070aa..ab1b4b7fc2f 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowCharSetTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowCharSetTest.java
@@ -76,6 +76,7 @@ public JdbcToArrowCharSetTest(Table table) {
    * @throws ClassNotFoundException on error
    */
   @Before
+  @Override
   public void setUp() throws SQLException, ClassNotFoundException {
     String url = "jdbc:h2:mem:JdbcToArrowTest?characterEncoding=UTF-8";
     String driver = "org.h2.Driver";
@@ -107,6 +108,7 @@ public static Collection<Object[]> getTestData() throws SQLException, ClassNotFo
    * the multi-byte CJK characters.
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     testDataSets(sqlToArrow(conn, table.getQuery(), new RootAllocator(Integer.MAX_VALUE),
         Calendar.getInstance()), false);
@@ -142,6 +144,7 @@ public void testJdbcSchemaMetadata() throws SQLException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     JdbcToArrowTestHelper.assertFieldMetadataIsEmpty(root);
 
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowDataTypesTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowDataTypesTest.java
index ae4fffd0f94..54e7d5ffb27 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowDataTypesTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowDataTypesTest.java
@@ -145,6 +145,7 @@ public static Collection<Object[]> getTestData() throws SQLException, ClassNotFo
    * Test Method to test JdbcToArrow Functionality for various H2 DB based datatypes.
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     testDataSets(sqlToArrow(conn, table.getQuery(), new RootAllocator(Integer.MAX_VALUE),
         Calendar.getInstance()), false);
@@ -185,6 +186,7 @@ public void testJdbcSchemaMetadata() throws SQLException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     JdbcToArrowTestHelper.assertFieldMetadataIsEmpty(root);
 
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowMapDataTypeTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowMapDataTypeTest.java
index 43862a93c39..a5d1ffa3f64 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowMapDataTypeTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowMapDataTypeTest.java
@@ -45,6 +45,7 @@ public JdbcToArrowMapDataTypeTest() throws IOException {
    * Test Method to test JdbcToArrow Functionality for Map form Types.OTHER column
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     Calendar calendar = Calendar.getInstance();
     ResultSetMetaData rsmd = getQueryMetaData(table.getQuery());
@@ -68,6 +69,7 @@ public void testJdbcToArrowValues() throws SQLException, IOException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     assertMapVectorValues((MapVector) root.getVector(MAP), table.getRowCount(),
             getMapValues(table.getValues(), MAP));
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowNullTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowNullTest.java
index 5731f27c5b3..31d32bd6489 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowNullTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowNullTest.java
@@ -124,6 +124,7 @@ public static Collection<Object[]> getTestData() throws SQLException, ClassNotFo
    * Test Method to test JdbcToArrow Functionality for various H2 DB based datatypes with null values.
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     testDataSets(sqlToArrow(conn, table.getQuery(), new RootAllocator(Integer.MAX_VALUE),
         Calendar.getInstance()), false);
@@ -168,6 +169,7 @@ public void testJdbcSchemaMetadata() throws SQLException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     JdbcToArrowTestHelper.assertFieldMetadataIsEmpty(root);
 
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowOptionalColumnsTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowOptionalColumnsTest.java
index eebcbe64c0e..4d0bbfc7a99 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowOptionalColumnsTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowOptionalColumnsTest.java
@@ -70,6 +70,7 @@ public static Collection<Object[]> getTestData() throws SQLException, ClassNotFo
    * Test Method to test JdbcToArrow Functionality for dealing with nullable and non-nullable columns.
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     testDataSets(sqlToArrow(conn, table.getQuery(), new RootAllocator(Integer.MAX_VALUE)), false);
   }
@@ -82,6 +83,7 @@ public void testJdbcToArrowValues() throws SQLException, IOException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     JdbcToArrowTestHelper.assertFieldMetadataIsEmpty(root);
 
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTest.java
index 7641fa7f165..a925dd7ee32 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTest.java
@@ -101,6 +101,7 @@ public static Collection<Object[]> getTestData() throws SQLException, ClassNotFo
    * Test Method to test JdbcToArrow Functionality for various H2 DB based datatypes with only one test data file.
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     testDataSets(sqlToArrow(conn, table.getQuery(), new RootAllocator(Integer.MAX_VALUE),
         Calendar.getInstance()), false);
@@ -149,6 +150,7 @@ public void testJdbcSchemaMetadata() throws SQLException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     JdbcToArrowTestHelper.assertFieldMetadataIsEmpty(root);
     assertBigIntVectorValues((BigIntVector) root.getVector(BIGINT), table.getRowCount(),
diff --git a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTimeZoneTest.java b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTimeZoneTest.java
index 462a75da514..fe08db161c8 100644
--- a/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTimeZoneTest.java
+++ b/java/adapter/jdbc/src/test/java/org/apache/arrow/adapter/jdbc/h2/JdbcToArrowTimeZoneTest.java
@@ -103,6 +103,7 @@ public static Collection<Object[]> getTestData() throws SQLException, ClassNotFo
    * Time and Timestamp datatype.
    */
   @Test
+  @Override
   public void testJdbcToArrowValues() throws SQLException, IOException {
     testDataSets(sqlToArrow(conn, table.getQuery(), new RootAllocator(Integer.MAX_VALUE),
         Calendar.getInstance(TimeZone.getTimeZone(table.getTimezone()))), false);
@@ -139,6 +140,7 @@ public void testJdbcSchemaMetadata() throws SQLException {
    * @param isIncludeMapVector is this dataset checks includes map column.
    *          Jdbc type to 'map' mapping declared in configuration only manually
    */
+  @Override
   public void testDataSets(VectorSchemaRoot root, boolean isIncludeMapVector) {
     JdbcToArrowTestHelper.assertFieldMetadataIsEmpty(root);
 
diff --git a/java/c/src/main/java/org/apache/arrow/c/Format.java b/java/c/src/main/java/org/apache/arrow/c/Format.java
index 2875e46f749..a5f44859e83 100644
--- a/java/c/src/main/java/org/apache/arrow/c/Format.java
+++ b/java/c/src/main/java/org/apache/arrow/c/Format.java
@@ -18,6 +18,7 @@
 package org.apache.arrow.c;
 
 import java.util.Arrays;
+import java.util.Locale;
 import java.util.stream.Collectors;
 
 import org.apache.arrow.util.Preconditions;
@@ -127,7 +128,7 @@ static String asString(ArrowType arrowType) {
                 String.format("Int type with bitwidth %d is unsupported", type.getBitWidth()));
         }
         if (type.getIsSigned()) {
-          format = format.toLowerCase();
+          format = format.toLowerCase(Locale.ROOT);
         }
         return format;
       }
diff --git a/java/c/src/main/java/org/apache/arrow/c/jni/CDataJniException.java b/java/c/src/main/java/org/apache/arrow/c/jni/CDataJniException.java
index bebd434f3db..df16839c78a 100644
--- a/java/c/src/main/java/org/apache/arrow/c/jni/CDataJniException.java
+++ b/java/c/src/main/java/org/apache/arrow/c/jni/CDataJniException.java
@@ -36,10 +36,10 @@ public int getErrno() {
   }
 
   @Override
-  public String toString() {
+  public String getMessage() {
     return "CDataJniException{" +
         "errno=" + errno +
-        ", message=" + getMessage() +
+        ", message=" + super.getMessage() +
         '}';
   }
 }
diff --git a/java/c/src/test/java/org/apache/arrow/c/DictionaryTest.java b/java/c/src/test/java/org/apache/arrow/c/DictionaryTest.java
index d892781756e..3416547fca3 100644
--- a/java/c/src/test/java/org/apache/arrow/c/DictionaryTest.java
+++ b/java/c/src/test/java/org/apache/arrow/c/DictionaryTest.java
@@ -23,6 +23,7 @@
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.nio.channels.Channels;
+import java.nio.charset.StandardCharsets;
 import java.util.Collections;
 
 import org.apache.arrow.c.ArrowArray;
@@ -95,9 +96,9 @@ public void testWithDictionary() throws Exception {
     // create dictionary and provider
     final VarCharVector dictVector = new VarCharVector("dict", allocator);
     dictVector.allocateNewSafe();
-    dictVector.setSafe(0, "aa".getBytes());
-    dictVector.setSafe(1, "bb".getBytes());
-    dictVector.setSafe(2, "cc".getBytes());
+    dictVector.setSafe(0, "aa".getBytes(StandardCharsets.UTF_8));
+    dictVector.setSafe(1, "bb".getBytes(StandardCharsets.UTF_8));
+    dictVector.setSafe(2, "cc".getBytes(StandardCharsets.UTF_8));
     dictVector.setValueCount(3);
 
     Dictionary dictionary = new Dictionary(dictVector, new DictionaryEncoding(0L, false, /* indexType= */null));
@@ -106,10 +107,10 @@ public void testWithDictionary() throws Exception {
     // create vector and encode it
     final VarCharVector vector = new VarCharVector("vector", allocator);
     vector.allocateNewSafe();
-    vector.setSafe(0, "bb".getBytes());
-    vector.setSafe(1, "bb".getBytes());
-    vector.setSafe(2, "cc".getBytes());
-    vector.setSafe(3, "aa".getBytes());
+    vector.setSafe(0, "bb".getBytes(StandardCharsets.UTF_8));
+    vector.setSafe(1, "bb".getBytes(StandardCharsets.UTF_8));
+    vector.setSafe(2, "cc".getBytes(StandardCharsets.UTF_8));
+    vector.setSafe(3, "aa".getBytes(StandardCharsets.UTF_8));
     vector.setValueCount(4);
 
     // get the encoded vector
@@ -163,11 +164,11 @@ private ArrowStreamReader createMultiBatchReader() throws IOException {
       // create dictionary and provider
       DictionaryProvider.MapDictionaryProvider provider = new DictionaryProvider.MapDictionaryProvider();
       dictVector.allocateNewSafe();
-      dictVector.setSafe(0, "aa".getBytes());
-      dictVector.setSafe(1, "bb".getBytes());
-      dictVector.setSafe(2, "cc".getBytes());
-      dictVector.setSafe(3, "dd".getBytes());
-      dictVector.setSafe(4, "ee".getBytes());
+      dictVector.setSafe(0, "aa".getBytes(StandardCharsets.UTF_8));
+      dictVector.setSafe(1, "bb".getBytes(StandardCharsets.UTF_8));
+      dictVector.setSafe(2, "cc".getBytes(StandardCharsets.UTF_8));
+      dictVector.setSafe(3, "dd".getBytes(StandardCharsets.UTF_8));
+      dictVector.setSafe(4, "ee".getBytes(StandardCharsets.UTF_8));
       dictVector.setValueCount(5);
       Dictionary dictionary = new Dictionary(dictVector, new DictionaryEncoding(0L, false, /* indexType= */null));
       provider.put(dictionary);
diff --git a/java/c/src/test/java/org/apache/arrow/c/NativeUtilTest.java b/java/c/src/test/java/org/apache/arrow/c/NativeUtilTest.java
index f46a0128c86..9a322b76379 100644
--- a/java/c/src/test/java/org/apache/arrow/c/NativeUtilTest.java
+++ b/java/c/src/test/java/org/apache/arrow/c/NativeUtilTest.java
@@ -64,7 +64,7 @@ public void testString() {
   @Test
   public void testToJavaArray() {
     long[] nativeArray = new long[] { 1, 2, 3 };
-    try (ArrowBuf buffer = allocator.buffer(Long.BYTES * nativeArray.length, null)) {
+    try (ArrowBuf buffer = allocator.buffer(Long.BYTES * ((long) nativeArray.length), null)) {
       for (long value : nativeArray) {
         buffer.writeLong(value);
       }
diff --git a/java/c/src/test/java/org/apache/arrow/c/RoundtripTest.java b/java/c/src/test/java/org/apache/arrow/c/RoundtripTest.java
index 768394ef7ab..8dcd2ff9a23 100644
--- a/java/c/src/test/java/org/apache/arrow/c/RoundtripTest.java
+++ b/java/c/src/test/java/org/apache/arrow/c/RoundtripTest.java
@@ -511,7 +511,7 @@ public void testNullVector() {
   @Test
   public void testVarBinaryVector() {
     try (final VarBinaryVector vector = new VarBinaryVector("v", allocator)) {
-      setVector(vector, "abc".getBytes(), "def".getBytes(), null);
+      setVector(vector, "abc".getBytes(StandardCharsets.UTF_8), "def".getBytes(StandardCharsets.UTF_8), null);
       assertTrue(roundtrip(vector, VarBinaryVector.class));
     }
   }
@@ -537,7 +537,7 @@ public void testLargeVarBinaryVector() {
 
       String str = "hello world";
       try (ArrowBuf buf = allocator.buffer(16)) {
-        buf.setBytes(0, str.getBytes());
+        buf.setBytes(0, str.getBytes(StandardCharsets.UTF_8));
         binHolder.start = 0;
         binHolder.end = str.length();
         binHolder.buffer = buf;
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationOutcomeDetails.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationOutcomeDetails.java
index 3ceda71cce0..fd534b18998 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationOutcomeDetails.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationOutcomeDetails.java
@@ -119,16 +119,14 @@ public boolean isAllocationFailed() {
 
     @Override
     public String toString() {
-      return new StringBuilder()
-          .append("allocator[" + accountant.getName() + "]")
-          .append(" reservation: " + accountant.getInitReservation())
-          .append(" limit: " + limit)
-          .append(" used: " + used)
-          .append(" requestedSize: " + requestedSize)
-          .append(" allocatedSize: " + allocatedSize)
-          .append(" localAllocationStatus: " + (allocationFailed ? "fail" : "success"))
-          .append("\n")
-          .toString();
+      return "allocator[" + accountant.getName() + "]" +
+          " reservation: " + accountant.getInitReservation() +
+          " limit: " + limit +
+          " used: " + used +
+          " requestedSize: " + requestedSize +
+          " allocatedSize: " + allocatedSize +
+          " localAllocationStatus: " + (allocationFailed ? "fail" : "success") +
+          "\n";
     }
   }
 
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationReservation.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationReservation.java
index c672dc48d79..cc6cbf7e6f2 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationReservation.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/AllocationReservation.java
@@ -84,5 +84,5 @@ public interface AllocationReservation extends AutoCloseable {
    */
   boolean isClosed();
 
-  void close();
+  @Override void close();
 }
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BaseAllocator.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BaseAllocator.java
index 189c800ba0f..89b8ffd322a 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BaseAllocator.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BaseAllocator.java
@@ -532,6 +532,8 @@ public String toVerboseString() {
     return sb.toString();
   }
 
+  /* Remove @SuppressWarnings after fixing https://github.com/apache/arrow/issues/41951 */
+  @SuppressWarnings("FormatStringAnnotation")
   private void hist(String noteFormat, Object... args) {
     if (historicalLog != null) {
       historicalLog.recordEvent(noteFormat, args);
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferLedger.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferLedger.java
index 62d268a1f44..c610066c982 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferLedger.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferLedger.java
@@ -58,6 +58,7 @@ boolean isOwningLedger() {
     return this == allocationManager.getOwningLedger();
   }
 
+  @Override
   public BufferAllocator getKey() {
     return allocator;
   }
@@ -419,7 +420,7 @@ public TransferResult transferOwnership(final ArrowBuf srcBuffer, final BufferAl
   /**
    * The outcome of a Transfer.
    */
-  public class TransferResult implements OwnershipTransferResult {
+  public static class TransferResult implements OwnershipTransferResult {
 
     // Whether this transfer fit within the target allocator's capacity.
     final boolean allocationFit;
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferManager.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferManager.java
index 6b622e71927..e7877d7a3e2 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferManager.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/BufferManager.java
@@ -49,5 +49,5 @@ public interface BufferManager extends AutoCloseable {
    */
   ArrowBuf getManagedBuffer(long size);
 
-  void close();
+  @Override void close();
 }
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/CheckAllocator.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/CheckAllocator.java
index dac4a3fcff5..bfbf1f212e6 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/CheckAllocator.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/CheckAllocator.java
@@ -60,6 +60,7 @@ static String check() {
     }
   }
 
+  @SuppressWarnings("URLEqualsHashCode")
   private static Set<URL> scanClasspath() {
     // LinkedHashSet appropriate here because it preserves insertion order
     // during iteration
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/ChildAllocator.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/ChildAllocator.java
index 67156f89d13..f3132cb46a2 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/ChildAllocator.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/ChildAllocator.java
@@ -21,7 +21,7 @@
 /**
  * Child allocator class. Only slightly different from the {@see RootAllocator},
  * in that these can't be created directly, but must be obtained from
- * {@see BufferAllocator#newChildAllocator(AllocatorOwner, long, long, int)}.
+ * {@link BufferAllocator#newChildAllocator(String, AllocationListener, long, long)}.
  *
  * <p>Child allocators can only be created by the root, or other children, so
  * this class is package private.</p>
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/LowCostIdentityHashMap.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/LowCostIdentityHashMap.java
index 740233ef411..251028aff0e 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/LowCostIdentityHashMap.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/LowCostIdentityHashMap.java
@@ -255,7 +255,7 @@ void rehash() {
   }
 
   private void computeMaxSize() {
-    threshold = (int) ((long) (elementData.length) * LOAD_FACTOR / 10000);
+    threshold = (int) ((long) elementData.length * LOAD_FACTOR / 10000);
   }
 
   /**
@@ -309,7 +309,6 @@ private void computeMaxSize() {
         elementData[index] = null;
       }
     }
-
     return (V) result;
   }
 
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/rounding/DefaultRoundingPolicy.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/rounding/DefaultRoundingPolicy.java
index 7ba231b0c20..5939e803fdc 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/rounding/DefaultRoundingPolicy.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/rounding/DefaultRoundingPolicy.java
@@ -44,20 +44,16 @@ public class DefaultRoundingPolicy implements RoundingPolicy {
 
   static {
     int defaultPageSize = Integer.getInteger("org.apache.memory.allocator.pageSize", 8192);
-    Throwable pageSizeFallbackCause = null;
     try {
       validateAndCalculatePageShifts(defaultPageSize);
     } catch (Throwable t) {
-      pageSizeFallbackCause = t;
       defaultPageSize = 8192;
     }
 
     int defaultMaxOrder = Integer.getInteger("org.apache.memory.allocator.maxOrder", 11);
-    Throwable maxOrderFallbackCause = null;
     try {
       validateAndCalculateChunkSize(defaultPageSize, defaultMaxOrder);
     } catch (Throwable t) {
-      maxOrderFallbackCause = t;
       defaultMaxOrder = 11;
     }
     DEFAULT_CHUNK_SIZE = validateAndCalculateChunkSize(defaultPageSize, defaultMaxOrder);
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/ArrowBufPointer.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/ArrowBufPointer.java
index b41576847d6..5775dd79434 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/ArrowBufPointer.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/ArrowBufPointer.java
@@ -27,7 +27,7 @@
  * Pointer to a memory region within an {@link ArrowBuf}.
  * It will be used as the basis for calculating hash code within a vector, and equality determination.
  */
-public final class ArrowBufPointer {
+public final class ArrowBufPointer implements Comparable<ArrowBufPointer> {
 
   /**
    * The hash code when the arrow buffer is null.
@@ -174,6 +174,7 @@ public int hashCode() {
    *     a positive integer if this pointer is larger;
    *     a negative integer if this pointer is smaller.
    */
+  @Override
   public int compareTo(ArrowBufPointer that) {
     if (this.buf == null || that.buf == null) {
       if (this.buf == null && that.buf == null) {
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/Float16.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/Float16.java
index 8040158fd09..5b80816d48f 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/Float16.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/Float16.java
@@ -161,7 +161,7 @@ public static float toFloat(short b) {
     int bits = b & 0xffff;
     int s = bits & SIGN_MASK;
     int e = (bits >>> EXPONENT_SHIFT) & SHIFTED_EXPONENT_MASK;
-    int m = (bits) & SIGNIFICAND_MASK;
+    int m = bits & SIGNIFICAND_MASK;
     int outE = 0;
     int outM = 0;
     if (e == 0) { // Denormal or 0
@@ -209,7 +209,7 @@ public static short toFloat16(float f) {
     int bits = Float.floatToRawIntBits(f);
     int s = (bits >>> FP32_SIGN_SHIFT);
     int e = (bits >>> FP32_EXPONENT_SHIFT) & FP32_SHIFTED_EXPONENT_MASK;
-    int m = (bits) & FP32_SIGNIFICAND_MASK;
+    int m = bits & FP32_SIGNIFICAND_MASK;
     int outE = 0;
     int outM = 0;
     if (e == 0xff) { // Infinite or NaN
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/HistoricalLog.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/HistoricalLog.java
index 21f063c939e..910cc1c21d7 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/HistoricalLog.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/HistoricalLog.java
@@ -17,8 +17,9 @@
 
 package org.apache.arrow.memory.util;
 
+import java.util.ArrayDeque;
 import java.util.Arrays;
-import java.util.LinkedList;
+import java.util.Deque;
 
 import org.checkerframework.checker.nullness.qual.Nullable;
 import org.slf4j.Logger;
@@ -30,7 +31,7 @@
  */
 public class HistoricalLog {
 
-  private final LinkedList<Event> history = new LinkedList<>();
+  private final Deque<Event> history = new ArrayDeque<>();
   private final String idString; // the formatted id string
   private final int limit; // the limit on the number of events kept
   private @Nullable Event firstEvent; // the first stack trace recorded
@@ -44,6 +45,8 @@ public class HistoricalLog {
    *                       associated with the object instance is best.
    * @param args           for the format string, or nothing if none are required
    */
+  @SuppressWarnings("FormatStringAnnotation")
+  /* Remove @SuppressWarnings after fixing https://github.com/apache/arrow/issues/41951 */
   public HistoricalLog(final String idStringFormat, Object... args) {
     this(Integer.MAX_VALUE, idStringFormat, args);
   }
@@ -66,7 +69,9 @@ public HistoricalLog(final String idStringFormat, Object... args) {
    *                       associated with the object instance is best.
    * @param args           for the format string, or nothing if none are required
    */
+  @SuppressWarnings("AnnotateFormatMethod")
   public HistoricalLog(final int limit, final String idStringFormat, Object... args) {
+    // Remove @SuppressWarnings after fixing https://github.com/apache/arrow/issues/41951
     this.limit = limit;
     this.idString = String.format(idStringFormat, args);
     this.firstEvent = null;
@@ -80,7 +85,9 @@ public HistoricalLog(final int limit, final String idStringFormat, Object... arg
    * @param noteFormat {@link String#format} format string that describes the event
    * @param args       for the format string, or nothing if none are required
    */
+  @SuppressWarnings("AnnotateFormatMethod")
   public synchronized void recordEvent(final String noteFormat, Object... args) {
+    // Remove @SuppressWarnings after fixing https://github.com/apache/arrow/issues/41951
     final String note = String.format(noteFormat, args);
     final Event event = new Event(note);
     if (firstEvent == null) {
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/MemoryUtil.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/MemoryUtil.java
index 2f74a985a3f..727e3531ee8 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/MemoryUtil.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/MemoryUtil.java
@@ -58,6 +58,7 @@ public class MemoryUtil {
   public static final boolean LITTLE_ENDIAN = ByteOrder.nativeOrder() == ByteOrder.LITTLE_ENDIAN;
 
   // Java 1.8, 9, 11, 17, 21 becomes 1, 9, 11, 17, and 21.
+  @SuppressWarnings("StringSplitter")
   private static final int majorVersion =
       Integer.parseInt(System.getProperty("java.specification.version").split("\\D+")[0]);
 
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/MurmurHasher.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/MurmurHasher.java
index 5de98d23bb8..9c5d0b90861 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/MurmurHasher.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/MurmurHasher.java
@@ -162,7 +162,7 @@ public boolean equals(@Nullable Object o) {
     if (this == o) {
       return true;
     }
-    if (o == null || getClass() != o.getClass()) {
+    if (!(o instanceof MurmurHasher)) {
       return false;
     }
     MurmurHasher that = (MurmurHasher) o;
diff --git a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/SimpleHasher.java b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/SimpleHasher.java
index 3bf3c2a8283..670129d3fb2 100644
--- a/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/SimpleHasher.java
+++ b/java/memory/memory-core/src/main/java/org/apache/arrow/memory/util/hash/SimpleHasher.java
@@ -53,6 +53,7 @@ protected SimpleHasher() {
    * @param length length of the memory region.
    * @return the hash code.
    */
+  @Override
   public int hashCode(long address, long length) {
     int hashValue = 0;
     int index = 0;
diff --git a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestArrowBuf.java b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestArrowBuf.java
index b4385b72a38..f01d152f84b 100644
--- a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestArrowBuf.java
+++ b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestArrowBuf.java
@@ -141,7 +141,7 @@ public void testSetBytesBigEndian() {
   }
 
   @Test
-  /**
+  /*
    * Test that allocation history is not recorded even though
    * assertions are enabled in tests (GH-34338).
    */
diff --git a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestBaseAllocator.java b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestBaseAllocator.java
index 535d5c15e89..1f9e65831b4 100644
--- a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestBaseAllocator.java
+++ b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestBaseAllocator.java
@@ -374,7 +374,7 @@ public void testCustomizedAllocationManager() {
       assertEquals(1, arrowBuf1.getInt(0));
 
       try {
-        final ArrowBuf arrowBuf2 = allocator.buffer(1);
+        allocator.buffer(1);
         fail("allocated memory beyond max allowed");
       } catch (OutOfMemoryException e) {
         // expected
@@ -1077,7 +1077,7 @@ public void testMemoryLeakWithReservation() throws Exception {
           "child2", 1024, MAX_ALLOCATION);
       rootAllocator.verify();
 
-      ArrowBuf buff = childAllocator2.buffer(256);
+      childAllocator2.buffer(256);
 
       Exception exception = assertThrows(IllegalStateException.class, () -> {
         childAllocator2.close();
diff --git a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestLowCostIdentityHashMap.java b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestLowCostIdentityHashMap.java
index 0cabc4a0571..234a6447ddb 100644
--- a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestLowCostIdentityHashMap.java
+++ b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/TestLowCostIdentityHashMap.java
@@ -147,7 +147,7 @@ public void testLargeMap() throws Exception {
     assertTrue(hashMap.isEmpty());
   }
 
-  private class StringWithKey implements ValueWithKeyIncluded<String> {
+  private static class StringWithKey implements ValueWithKeyIncluded<String> {
 
     private String myValue;
     private String myKey;
diff --git a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestArrowBufPointer.java b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestArrowBufPointer.java
index a1d5624a7e8..04e588ed16f 100644
--- a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestArrowBufPointer.java
+++ b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestArrowBufPointer.java
@@ -55,16 +55,16 @@ public void testArrowBufPointersEqual() {
     try (ArrowBuf buf1 = allocator.buffer(BUFFER_LENGTH);
         ArrowBuf buf2 = allocator.buffer(BUFFER_LENGTH)) {
       for (int i = 0; i < BUFFER_LENGTH / 4; i++) {
-        buf1.setInt(i * 4, i * 1234);
-        buf2.setInt(i * 4, i * 1234);
+        buf1.setInt(i * 4L, i * 1234);
+        buf2.setInt(i * 4L, i * 1234);
       }
 
       ArrowBufPointer ptr1 = new ArrowBufPointer(null, 0, 100);
       ArrowBufPointer ptr2 = new ArrowBufPointer(null, 100, 5032);
       assertTrue(ptr1.equals(ptr2));
       for (int i = 0; i < BUFFER_LENGTH / 4; i++) {
-        ptr1.set(buf1, i * 4, 4);
-        ptr2.set(buf2, i * 4, 4);
+        ptr1.set(buf1, i * 4L, 4);
+        ptr2.set(buf2, i * 4L, 4);
         assertTrue(ptr1.equals(ptr2));
       }
     }
@@ -76,8 +76,8 @@ public void testArrowBufPointersHashCode() {
     try (ArrowBuf buf1 = allocator.buffer(vectorLength * 4);
          ArrowBuf buf2 = allocator.buffer(vectorLength * 4)) {
       for (int i = 0; i < vectorLength; i++) {
-        buf1.setInt(i * 4, i);
-        buf2.setInt(i * 4, i);
+        buf1.setInt(i * 4L, i);
+        buf2.setInt(i * 4L, i);
       }
 
       CounterHasher hasher1 = new CounterHasher();
@@ -90,8 +90,8 @@ public void testArrowBufPointersHashCode() {
       assertEquals(ArrowBufPointer.NULL_HASH_CODE, pointer2.hashCode());
 
       for (int i = 0; i < vectorLength; i++) {
-        pointer1.set(buf1, i * 4, 4);
-        pointer2.set(buf2, i * 4, 4);
+        pointer1.set(buf1, i * 4L, 4);
+        pointer2.set(buf2, i * 4L, 4);
 
         assertEquals(pointer1.hashCode(), pointer2.hashCode());
 
@@ -188,7 +188,7 @@ public void testArrowBufPointersComparison() {
    * Hasher with a counter that increments each time a hash code is calculated.
    * This is to validate that the hash code in {@link ArrowBufPointer} is reused.
    */
-  class CounterHasher implements ArrowBufHasher {
+  static class CounterHasher implements ArrowBufHasher {
 
     protected int counter = 0;
 
@@ -211,7 +211,7 @@ public int hashCode() {
 
     @Override
     public boolean equals(Object o) {
-      return o != null && this.getClass() == o.getClass();
+      return o instanceof CounterHasher;
     }
   }
 }
diff --git a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestByteFunctionHelpers.java b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestByteFunctionHelpers.java
index 04a715962df..7a44a5f2d72 100644
--- a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestByteFunctionHelpers.java
+++ b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/TestByteFunctionHelpers.java
@@ -19,6 +19,8 @@
 
 import static org.junit.Assert.assertEquals;
 
+import java.nio.charset.StandardCharsets;
+
 import org.apache.arrow.memory.ArrowBuf;
 import org.apache.arrow.memory.BufferAllocator;
 import org.apache.arrow.memory.RootAllocator;
@@ -121,9 +123,9 @@ public void testStringCompare() {
       String rightStr = rightStrings[i];
 
       ArrowBuf left = allocator.buffer(SIZE);
-      left.setBytes(0, leftStr.getBytes());
+      left.setBytes(0, leftStr.getBytes(StandardCharsets.UTF_8));
       ArrowBuf right = allocator.buffer(SIZE);
-      right.setBytes(0, rightStr.getBytes());
+      right.setBytes(0, rightStr.getBytes(StandardCharsets.UTF_8));
 
       assertEquals(leftStr.compareTo(rightStr) < 0 ? -1 : 1,
           ByteFunctionHelpers.compare(left, 0, leftStr.length(), right, 0, rightStr.length()));
diff --git a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/hash/TestArrowBufHasher.java b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/hash/TestArrowBufHasher.java
index 3da0602bdfd..cc5ce49e548 100644
--- a/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/hash/TestArrowBufHasher.java
+++ b/java/memory/memory-core/src/test/java/org/apache/arrow/memory/util/hash/TestArrowBufHasher.java
@@ -66,8 +66,8 @@ public void testHasher() {
          ArrowBuf buf2 = allocator.buffer(BUFFER_LENGTH)) {
       // prepare data
       for (int i = 0; i < BUFFER_LENGTH / 4; i++) {
-        buf1.setFloat(i * 4, i / 10.0f);
-        buf2.setFloat(i * 4, i / 10.0f);
+        buf1.setFloat(i * 4L, i / 10.0f);
+        buf2.setFloat(i * 4L, i / 10.0f);
       }
 
       verifyHashCodesEqual(buf1, 0, 100, buf2, 0, 100);
@@ -95,7 +95,7 @@ public void testHasherNegative() {
     try (ArrowBuf buf = allocator.buffer(BUFFER_LENGTH)) {
       // prepare data
       for (int i = 0; i < BUFFER_LENGTH / 4; i++) {
-        buf.setFloat(i * 4, i / 10.0f);
+        buf.setFloat(i * 4L, i / 10.0f);
       }
 
       assertThrows(IllegalArgumentException.class, () -> {
@@ -120,13 +120,13 @@ public void testHasherLessThanInt() {
       buf2.writeBytes("bar2".getBytes(StandardCharsets.UTF_8));
 
       for (int i = 1; i <= 4; i ++) {
-        verifyHashCodeNotEqual(buf1, 0, i, buf2, 0, i);
+        verifyHashCodeNotEqual(buf1, i, buf2, i);
       }
     }
   }
 
-  private void verifyHashCodeNotEqual(ArrowBuf buf1, int offset1, int length1,
-                                      ArrowBuf buf2, int offset2, int length2) {
+  private void verifyHashCodeNotEqual(ArrowBuf buf1, int length1,
+                                      ArrowBuf buf2, int length2) {
     int hashCode1 = hasher.hashCode(buf1, 0, length1);
     int hashCode2 = hasher.hashCode(buf2, 0, length2);
     assertNotEquals(hashCode1, hashCode2);
diff --git a/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/NettyArrowBuf.java b/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/NettyArrowBuf.java
index 466444c7d53..ae1f30a8684 100644
--- a/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/NettyArrowBuf.java
+++ b/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/NettyArrowBuf.java
@@ -413,7 +413,7 @@ protected int _getUnsignedMedium(int index) {
   protected int _getUnsignedMediumLE(int index) {
     this.chk(index, 3);
     long addr = this.addr(index);
-    return PlatformDependent.getByte(addr) & 255 |
+    return (PlatformDependent.getByte(addr) & 255) |
         (Short.reverseBytes(PlatformDependent.getShort(addr + 1L)) & '\uffff') << 8;
   }
 
diff --git a/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java b/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java
index e51c6c3d488..ab0cd0c9e6e 100644
--- a/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java
+++ b/java/memory/memory-netty-buffer-patch/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java
@@ -36,19 +36,12 @@ public class UnsafeDirectLittleEndian extends WrappedByteBuf {
   private final AbstractByteBuf wrapped;
   private final long memoryAddress;
 
-  UnsafeDirectLittleEndian(DuplicatedByteBuf buf) {
-    this(buf, true);
-  }
-
-  UnsafeDirectLittleEndian(LargeBuffer buf) {
-    this(buf, true);
-  }
-
-  UnsafeDirectLittleEndian(PooledUnsafeDirectByteBuf buf) {
-    this(buf, true);
-  }
-
-  private UnsafeDirectLittleEndian(AbstractByteBuf buf, boolean fake) {
+  /**
+   * Constructs a new instance.
+   *
+   * @param buf The buffer to wrap
+   */
+  public UnsafeDirectLittleEndian(AbstractByteBuf buf) {
     super(buf);
 
     this.wrapped = buf;
diff --git a/java/memory/memory-netty-buffer-patch/src/test/java/io/netty/buffer/TestUnsafeDirectLittleEndian.java b/java/memory/memory-netty-buffer-patch/src/test/java/io/netty/buffer/TestUnsafeDirectLittleEndian.java
index 043c2c1605a..4717e48f27b 100644
--- a/java/memory/memory-netty-buffer-patch/src/test/java/io/netty/buffer/TestUnsafeDirectLittleEndian.java
+++ b/java/memory/memory-netty-buffer-patch/src/test/java/io/netty/buffer/TestUnsafeDirectLittleEndian.java
@@ -23,7 +23,6 @@
 import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
-import java.nio.ByteOrder;
 import java.nio.charset.StandardCharsets;
 
 import org.junit.Test;
@@ -34,9 +33,9 @@
 import io.netty.buffer.UnsafeDirectLittleEndian;
 
 public class TestUnsafeDirectLittleEndian {
-  private static final boolean LITTLE_ENDIAN = ByteOrder.nativeOrder() == ByteOrder.LITTLE_ENDIAN;
 
   @Test
+  @SuppressWarnings("CatchAndPrintStackTrace")
   public void testPrimitiveGetSet() {
     ByteBuf byteBuf = Unpooled.directBuffer(64);
     UnsafeDirectLittleEndian unsafeDirect = new UnsafeDirectLittleEndian(new LargeBuffer(byteBuf));
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 58354d0c2ee..1e4e06df7e9 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 a782523cbc6..0c99062021f 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 a6da36bb35a..792ae53a940 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() {
     ListAppender<ILoggingEvent> memoryLogsAppender = new ListAppender<>();
     memoryLogsAppender.list = Collections.synchronizedList(memoryLogsAppender.list);
diff --git a/java/pom.xml b/java/pom.xml
index 9be9d431d47..a59d29c5763 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -95,6 +95,8 @@
     <checkstyle.failOnViolation>true</checkstyle.failOnViolation>
     <errorprone.javac.version>9+181-r4173-1</errorprone.javac.version>
     <error_prone_core.version>2.24.0</error_prone_core.version>
+    <error_prone_annotations.version.jdk11>2.24.0</error_prone_annotations.version.jdk11>
+    <error_prone_annotations.version.jdk8>2.10.0</error_prone_annotations.version.jdk8>
     <maven-compiler-plugin.version>3.12.1</maven-compiler-plugin.version>
     <mockito.core.version>5.11.0</mockito.core.version>
     <mockito.inline.version>5.2.0</mockito.inline.version>
@@ -995,7 +997,7 @@
               <encoding>UTF-8</encoding>
               <compilerArgs combine.children="append">
                 <arg>-XDcompilePolicy=simple</arg>
-                <arg>-Xplugin:ErrorProne -XepExcludedPaths:.*/(target/generated-sources)/.*</arg>
+                <arg>-Xplugin:ErrorProne -XepExcludedPaths:.*/(target/generated-source|format/src/main/java/org/apache/arrow/flatbuf)/.*</arg>
                 <arg>-J--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED</arg>
                 <arg>-J--add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED</arg>
                 <arg>-J--add-exports=jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED</arg>