[GitHub] [arrow] lidavidm commented on pull request #7685: ARROW-9362: [Java] Support reading/writing V4 MetadataVersion

2020-07-10 Thread GitBox
lidavidm commented on pull request #7685: URL: https://github.com/apache/arrow/pull/7685#issuecomment-656949942 > So is this ok to merge before #7290 ? This should be OK. It doesn't enable unions in the integration tests and prevents reading/writing unions with the old version. (Of

[GitHub] [arrow] lidavidm commented on pull request #7685: ARROW-9362: [Java] Support reading/writing V4 MetadataVersion

2020-07-10 Thread GitBox
lidavidm commented on pull request #7685: URL: https://github.com/apache/arrow/pull/7685#issuecomment-656738197 Would any Java maintainers be able to review? There's a bit of a circular dependency with this and #7290 - this is needed so that C++ will see the V5 metadata version and read