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 617f1a85 fix: don't require metadata order in
nanoarrow_ipc_integration (#589)
617f1a85 is described below
commit 617f1a8570ac6d8efee5b943f7f74830f6790a60
Author: Benjamin Kietzman <[email protected]>
AuthorDate: Fri Aug 16 19:26:28 2024 -0500
fix: don't require metadata order in nanoarrow_ipc_integration (#589)
Minor patch to the integration executable; Java produces IPC files with
potentially reordered metadata:
https://github.com/apache/arrow/actions/runs/10410485727/job/28832431571?pr=43715#step:9:9093
---
src/nanoarrow/integration/ipc_integration.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/nanoarrow/integration/ipc_integration.cc
b/src/nanoarrow/integration/ipc_integration.cc
index 52392c8d..b2a092bd 100644
--- a/src/nanoarrow/integration/ipc_integration.cc
+++ b/src/nanoarrow/integration/ipc_integration.cc
@@ -257,6 +257,7 @@ ArrowErrorCode Validate(struct ArrowError* error) {
NANOARROW_RETURN_NOT_OK(arrow_table.FromIpcFile(arrow_path, error));
nanoarrow::testing::TestingJSONComparison comparison;
+ comparison.set_compare_metadata_order(false);
NANOARROW_RETURN_NOT_OK(
comparison.CompareSchema(arrow_table.schema.get(),
json_table.schema.get(), error));
if (comparison.num_differences() != 0) {
