scampi commented on a change in pull request #6402:
URL: https://github.com/apache/arrow/pull/6402#discussion_r439778366



##########
File path: 
java/vector/src/test/java/org/apache/arrow/vector/TestValueVector.java
##########
@@ -886,6 +886,65 @@ public void testNullableFixedType4() {
    *  -- VarBinaryVector
    */
 
+  @Test /* VarCharVector */
+  public void testSplitAndTransfer1() {
+    try (final VarCharVector target = newVarCharVector("split-target", 
allocator)) {
+      try (final VarCharVector vector = newVarCharVector(EMPTY_SCHEMA_PATH, 
allocator)) {
+        vector.allocateNew(1024 * 10, 1024);
+
+        vector.set(0, STR1);
+        vector.set(1, STR2);
+        vector.set(2, STR3);
+        vector.setValueCount(3);
+
+        final long allocatedMem = allocator.getAllocatedMemory();
+        final int validityRefCnt = vector.getValidityBuffer().refCnt();
+        final int offsetRefCnt = vector.getOffsetBuffer().refCnt();
+        final int dataRefCnt = vector.getDataBuffer().refCnt();
+
+        // split and transfer with slice starting at the beginning: this 
should not allocate anything new
+        vector.splitAndTransferTo(0, 2, target);
+        assertEquals(allocator.getAllocatedMemory(), allocatedMem);
+        // 2 = validity and offset buffers are stored in the same arrowbuf
+        assertEquals(vector.getValidityBuffer().refCnt(), validityRefCnt + 2);

Review comment:
       Not at all, that's a good question. I've improved the assertion's 
comment in commit 2e13c5239137e54b0cd128240e6a3d0895c0c36f. Do they make sense ?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to