paleolimbot commented on PR #41765: URL: https://github.com/apache/arrow/pull/41765#issuecomment-2738258514
Apologies for taking a while to circle back here...at the time I started this PR I didn't have a great grasp on how all the pieces of the Parquet testing went together and it all floated back in today. I made https://github.com/apache/parquet-testing/pull/72 to ensure we have a file to test against, have tested it against these changes, and I can add update the submodule + add tests here if this/the PR into parquet-testing is something the community is interested in. Given that there are three new types forthcoming in Parquet (and I'm not sure the Geometry/Geography implementation will make it in before the April 1 feature freeze), I think this would be great to get in to the next version if there's review bandwidth! > Is there a reason to make this behavior optional instead of standard? I'm happy to do either...I personally think that making the "ignore unknown logical types" the default behaviour would be best. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
