Skip to content

Commit 983dd8c

Browse files
committed
MINOR: Fix TestVariantVector#testGetObject for parquet-variant 1.18.1
parquet-variant 1.18.1 added eager validation of the variant metadata header in the Variant constructor. The test's placeholder metadata bytes no longer decode as a structurally valid header, so getObject() throws IllegalArgumentException. Use a valid single-byte (version 1, empty dictionary) metadata header instead.
1 parent 671f71f commit 983dd8c

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

arrow-variant/src/test/java/org/apache/arrow/variant/extension/TestVariantVector.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -769,7 +769,9 @@ void testGetObject() {
769769
ArrowBuf metadataBuf = allocator.buffer(10);
770770
ArrowBuf valueBuf = allocator.buffer(10)) {
771771

772-
byte[] metadata = new byte[] {1, 2};
772+
// Version 1, empty dictionary (single header byte) - a structurally valid variant
773+
// metadata, since getObject() eagerly parses the metadata header.
774+
byte[] metadata = new byte[] {1};
773775
byte[] value = new byte[] {3, 4, 5};
774776
metadataBuf.setBytes(0, metadata);
775777
valueBuf.setBytes(0, value);

0 commit comments

Comments
 (0)