albertlockett commented on PR #8005: URL: https://github.com/apache/arrow-rs/pull/8005#issuecomment-3127888914
> This seems like a good idea to me. Thank you @albertlockett > > I think it would be nice to: > > 1. Test the other way (write dictionary first then string) > 2. Document the behavior somewhere in the arrow writers doc (that you can write batches that are logically equal) > > It might also be worth considering (in a future PR) the definition of compatible -- like String and LargeString for example, or String, LargeString, and StringView Thanks @alamb! Made these changes. I created an issue to track the future work to improve the type compatibility: https://github.com/apache/arrow-rs/issues/8012 -- 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
