mapleFU commented on PR #34511: URL: https://github.com/apache/arrow/pull/34511#issuecomment-1634315378
Maybe we can rebase and change ` bool CompatibleType(ConvertedType::type converted_type) const;` from schema to `stream_writer` and `stream_reader` in this patch, previously I think introduce a rule breaks the format of parquet, and introduce some schema inconsistent risk. But now I think `stream_writer` is already bad enough, adding a rule would not make it worse. -- 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]
