[GitHub] [arrow] wesm commented on pull request #7664: ARROW-9265: [C++] Allow writing and reading V4-compliant IPC data

2020-07-09 Thread GitBox
wesm commented on pull request #7664: URL: https://github.com/apache/arrow/pull/7664#issuecomment-656341760 I'm quickly taking care of these small things so this can be merged This is an automated message from the Apache Git

[GitHub] [arrow] wesm commented on pull request #7664: ARROW-9265: [C++] Allow writing and reading V4-compliant IPC data

2020-07-08 Thread GitBox
wesm commented on pull request #7664: URL: https://github.com/apache/arrow/pull/7664#issuecomment-655536712 I just sent an e-mail to the ML. We don't want to continue producing V4 metadata unless we need to for forward compatibility reasons. --

[GitHub] [arrow] wesm commented on pull request #7664: ARROW-9265: [C++] Allow writing and reading V4-compliant IPC data

2020-07-08 Thread GitBox
wesm commented on pull request #7664: URL: https://github.com/apache/arrow/pull/7664#issuecomment-655530738 @lidavidm @pitrou I haven't looked at the patch yet, but V5 must be the default produced by writers but V4 can be opted in to. -