Skip to content

Commit 47129f5

Browse files
committed
Fix test cases to make sure that the exported vectors have 0-capacity offset buffer
1 parent 6d04679 commit 47129f5

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

c/src/test/java/org/apache/arrow/c/RoundtripTest.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@
7979
import org.apache.arrow.vector.ValueVector;
8080
import org.apache.arrow.vector.VarBinaryVector;
8181
import org.apache.arrow.vector.VarCharVector;
82-
import org.apache.arrow.vector.VariableWidthVector;
8382
import org.apache.arrow.vector.VectorSchemaRoot;
8483
import org.apache.arrow.vector.ViewVarBinaryVector;
8584
import org.apache.arrow.vector.ViewVarCharVector;
@@ -185,7 +184,8 @@ boolean roundtrip(FieldVector vector, Class<?> clazz) {
185184
String.format("expected %s but was %s", clazz, imported.getClass()));
186185
result = VectorEqualsVisitor.vectorEquals(vector, imported);
187186

188-
if (imported instanceof BaseVariableWidthVector || imported instanceof BaseLargeVariableWidthVector) {
187+
if (imported instanceof BaseVariableWidthVector
188+
|| imported instanceof BaseLargeVariableWidthVector) {
189189
ArrowBuf offsetBuffer = imported.getOffsetBuffer();
190190
assertTrue(offsetBuffer.capacity() > 0);
191191
assertEquals(0, offsetBuffer.getInt(0));
@@ -614,7 +614,6 @@ public void testVarCharVector() {
614614
@Test
615615
public void testEmptyVarCharVector() {
616616
try (final VarCharVector vector = new VarCharVector("v", allocator)) {
617-
setVector(vector, new String[] {});
618617
assertTrue(roundtrip(vector, VarCharVector.class));
619618
}
620619
}
@@ -655,7 +654,6 @@ public void testLargeVarCharVector() {
655654
@Test
656655
public void testEmptyLargeVarCharVector() {
657656
try (final LargeVarCharVector vector = new LargeVarCharVector("v", allocator)) {
658-
setVector(vector, new String[] {});
659657
assertTrue(roundtrip(vector, LargeVarCharVector.class));
660658
}
661659
}

0 commit comments

Comments
 (0)