This is an automated email from the ASF dual-hosted git repository.
apitrou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new 01bc1bd979 GH-48176: [C++][Parquet] Fix
arrow-ipc-message-internal-test failure (#48166)
01bc1bd979 is described below
commit 01bc1bd979f51a6f77013855c3194159f9a87f0b
Author: Vishwanatha-HD <[email protected]>
AuthorDate: Mon Dec 8 21:35:20 2025 +0530
GH-48176: [C++][Parquet] Fix arrow-ipc-message-internal-test failure
(#48166)
**Rationale for this change**
Fix `arrow-ipc-message-internal-test` on big endian systems by passing
endianness explicitly.
**Are these changes tested?**
Yes. The changes are tested on s390x arch to make sure things are working
fine. The fix is also tested on x86 arch, to make sure there is no new
regression introduced.
**Are there any user-facing changes?**
No
* GitHub main Issue link: #48151
* GitHub Issue: #48176
Lead-authored-by: Vishwanatha-HD <[email protected]>
Co-authored-by: Antoine Pitrou <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
---
cpp/src/arrow/ipc/message_internal_test.cc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/cpp/src/arrow/ipc/message_internal_test.cc
b/cpp/src/arrow/ipc/message_internal_test.cc
index 1114f55517..87ce3de39c 100644
--- a/cpp/src/arrow/ipc/message_internal_test.cc
+++ b/cpp/src/arrow/ipc/message_internal_test.cc
@@ -47,7 +47,7 @@ TEST(TestMessageInternal, TestByteIdentical) {
std::vector<std::shared_ptr<Field>> fields = {f0, f1};
std::shared_ptr<KeyValueMetadata> metadata =
KeyValueMetadata::Make({"key_1", "key_2"}, {"key_1_value",
"key_2_value"});
- auto schema = ::arrow::schema({f0}, metadata);
+ auto schema = ::arrow::schema({f0}, Endianness::Little, metadata);
// Serialize the Schema to a Buffer
std::shared_ptr<Buffer> out_buffer;
@@ -73,6 +73,7 @@ TEST(TestMessageInternal, TestByteIdentical) {
0x00, 0x00, 0x00, 0x00, 0x02, 0x00, 0x00, 0x00, 0x66, 0x30, 0x00, 0x00,
0x08, 0x00,
0x0C, 0x00, 0x08, 0x00, 0x07, 0x00, 0x08, 0x00, 0x00, 0x00, 0x00, 0x00,
0x00, 0x01,
0x40, 0x00, 0x00, 0x00};
+
Buffer expected_buffer(expected, sizeof(expected));
AssertBufferEqual(expected_buffer, *out_buffer);