mapleFU commented on PR #34511: URL: https://github.com/apache/arrow/pull/34511#issuecomment-1493374263
Hi @omrimallis . Personally I'm ok for this patch. If you want, you can encapsulate in in common header for `stream_reader.cc` and `stream_writer.cc`. My view is that, for reader, we need to support some backward-capability rules. However, for writer, I don't think we should keep the rule, because writer schema is created by user, and user can deduce specifict convert types. If you want, I think just put it to stream_reader and stream_writer is ok -- 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]
