fix: addressing reviews 11: minor changes to allocation logic

vibhatha · May 14, 2024 · 0089c14 · 0089c14
1 parent 694420e
commit 0089c14
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/java/vector/src/main/java/org/apache/arrow/vector/complex/BaseRepeatedValueViewVector.java b/java/vector/src/main/java/org/apache/arrow/vector/complex/BaseRepeatedValueViewVector.java
@@ -99,20 +99,15 @@ public boolean allocateNewSafe() {
   }
 
   private void allocateBuffers() {
-    offsetBuffer = allocateBuffers(offsetAllocationSizeInBytes, true);
-    sizeBuffer = allocateBuffers(sizeAllocationSizeInBytes, false);
+    offsetBuffer = allocateBuffers(offsetAllocationSizeInBytes);
+    sizeBuffer = allocateBuffers(sizeAllocationSizeInBytes);
   }
 
-  private ArrowBuf allocateBuffers(final long size, boolean isOffSet) {
+  private ArrowBuf allocateBuffers(final long size) {
     final int curSize = (int) size;
     ArrowBuf buffer = allocator.buffer(curSize);
     buffer.readerIndex(0);
     buffer.setZero(0, buffer.capacity());
-    if (isOffSet) {
-      offsetAllocationSizeInBytes = curSize;
-    } else {
-      sizeAllocationSizeInBytes = curSize;
-    }
     return buffer;
   }