This is an automated email from the ASF dual-hosted git repository.
paleolimbot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-nanoarrow.git
The following commit(s) were added to refs/heads/main by this push:
new 33476d68 fix(ci): Set endianness of decoder in dictionary batch decode
test (#878)
33476d68 is described below
commit 33476d686566c09018b3cec6cee8167a0f30b187
Author: Dewey Dunnington <[email protected]>
AuthorDate: Mon Apr 20 09:03:02 2026 -0500
fix(ci): Set endianness of decoder in dictionary batch decode test (#878)
The dictionary batch decode was failing on big endian CI because we need
to explicitly set the endianness of the decoder to ensure it byte swaps
buffers correctly.
---
src/nanoarrow/ipc/decoder_test.cc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/nanoarrow/ipc/decoder_test.cc
b/src/nanoarrow/ipc/decoder_test.cc
index 2143b610..48ed0af1 100644
--- a/src/nanoarrow/ipc/decoder_test.cc
+++ b/src/nanoarrow/ipc/decoder_test.cc
@@ -699,7 +699,7 @@ TEST(NanoarrowIpcTest,
NanoarrowIpcDictionaryEncodingsUniqueIds) {
ArrowIpcDictionaryEncodingsReset(&dictionary_encodings);
}
-TEST(NanoarrowIpcTest, NanoarrowIpcDecodeDictionaryBatchDecode) {
+TEST(NanoarrowIpcTest, NanoarrowIpcDecodeDictionaryBatch) {
using namespace nanoarrow::literals;
struct ArrowIpcDictionaryEncodings dictionary_encodings;
@@ -734,6 +734,9 @@ TEST(NanoarrowIpcTest,
NanoarrowIpcDecodeDictionaryBatchDecode) {
// Check that we can't decode a dictionary batch if we haven't read a
dictionary batch
// message
ASSERT_EQ(ArrowIpcDecoderInit(&decoder), NANOARROW_OK);
+ ASSERT_EQ(ArrowIpcDecoderSetEndianness(&decoder,
NANOARROW_IPC_ENDIANNESS_LITTLE),
+ NANOARROW_OK);
+
struct ArrowBufferView body;
body.data.data = nullptr;
body.size_bytes = 0;