adderss comments

Author: LuciferYang

vector/src/main/java/org/apache/arrow/vector/complex/LargeListVector.java

@@ -1041,8 +1041,8 @@ public void setValueCount(int valueCount) {
      * TODO: revisit when 64-bit vectors are supported
      */
     Preconditions.checkArgument(
-        childValueCount <= Integer.MAX_VALUE && childValueCount >= 0,
-        "LargeListVector doesn't yet support 64-bit allocations: %s",
+        childValueCount >= 0 && childValueCount <= Integer.MAX_VALUE,
+        "LargeListVector childValueCount must be in [0, Integer.MAX_VALUE] but was: %s",
         childValueCount);
     vector.setValueCount((int) childValueCount);
   }

vector/src/test/java/org/apache/arrow/vector/TestLargeListVector.java

@@ -23,13 +23,15 @@
 import static org.junit.jupiter.api.Assertions.assertSame;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.fail;
 
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
 import java.util.UUID;
 import org.apache.arrow.memory.ArrowBuf;
 import org.apache.arrow.memory.BufferAllocator;
+import org.apache.arrow.memory.OutOfMemoryException;
 import org.apache.arrow.vector.complex.BaseRepeatedValueVector;
 import org.apache.arrow.vector.complex.LargeListVector;
 import org.apache.arrow.vector.complex.ListVector;
@@ -43,6 +45,7 @@
 import org.apache.arrow.vector.types.pojo.ArrowType;
 import org.apache.arrow.vector.types.pojo.Field;
 import org.apache.arrow.vector.types.pojo.FieldType;
+import org.apache.arrow.vector.util.OversizedAllocationException;
 import org.apache.arrow.vector.util.TransferPair;
 import org.apache.arrow.vector.util.UuidUtility;
 import org.junit.jupiter.api.AfterEach;
@@ -1200,14 +1203,13 @@ public void testSetValueCountAcceptsMaxIntChildValueCount() {
       // The Preconditions check should accept this value (not throw IllegalArgumentException).
       // The child vector may throw OversizedAllocationException due to memory limits,
       // which is expected and unrelated to the precondition validation.
-      vector.getOffsetBuffer().setLong(LargeListVector.OFFSET_WIDTH, (long) Integer.MAX_VALUE);
+      vector.getOffsetBuffer().setLong(LargeListVector.OFFSET_WIDTH, Integer.MAX_VALUE);
       vector.setLastSet(0);
       try {
         vector.setValueCount(1);
       } catch (IllegalArgumentException e) {
-        throw new AssertionError(
-            "setValueCount should not reject childValueCount = Integer.MAX_VALUE", e);
-      } catch (Exception e) {
+        fail("setValueCount should not reject childValueCount = Integer.MAX_VALUE", e);
+      } catch (OversizedAllocationException | OutOfMemoryException e) {
         // OversizedAllocationException or other allocation errors are expected
         // when trying to allocate Integer.MAX_VALUE elements — this is fine,
         // the precondition check itself passed.