[GitHub] [arrow] jmgpeeters commented on pull request #9629: ARROW-11838: [C++] Support IPC reads with shared dictionaries.

jmgpeeters commented on pull request #9629: URL: https://github.com/apache/arrow/pull/9629#issuecomment-809392482 Yeah, looked spurious to me, and the errors seemed unrelated to the changes. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [arrow] jmgpeeters commented on pull request #9629: ARROW-11838: [C++] Support IPC reads with shared dictionaries.

jmgpeeters commented on pull request #9629: URL: https://github.com/apache/arrow/pull/9629#issuecomment-806093450 @pitrou Yep, this is now ready for review. Was just letting it run through all the integration tests. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [arrow] jmgpeeters commented on pull request #9629: ARROW-11838: [C++] Support IPC reads with shared dictionaries.

jmgpeeters commented on pull request #9629: URL: https://github.com/apache/arrow/pull/9629#issuecomment-801261388 Hm, for some reason the github integration test checked out an older version of the arrow-testing data, Run ci/scripts/util_checkout.sh Submodule

[GitHub] [arrow] jmgpeeters commented on pull request #9629: ARROW-11838: [C++] Support IPC reads with shared dictionaries.

jmgpeeters commented on pull request #9629: URL: https://github.com/apache/arrow/pull/9629#issuecomment-790592214 To be clear, these test failures should be due to https://github.com/apache/arrow-testing/pull/59 not being merged yet - which provides the associated materialised test data.