paleolimbot commented on PR #41765: URL: https://github.com/apache/arrow/pull/41765#issuecomment-2483647131
> I'm just noticing this PR. Is there a reason to make this behavior optional instead of standard? I don't have an opinion here, although I think that the `arrow.opaque` (vendor: Parquet, type: Unknown Logical type with ID XXX) extension type makes it easier to make the case for having this be the default. Without that, there is a possibly good argument for erroring to avoid silently discarding logical type information. I ran out of time figuring out how to generate a Parquet file I could use to test the Python bindings here, but I'm happy to resurrect this PR and give it another shot if there is interest. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
