kumarlokesh commented on PR #7434: URL: https://github.com/apache/arrow-rs/pull/7434#issuecomment-2889163806
@alamb updated this PR to remove `"2. Support for Map"`. The sole focus of this PR is `"3. Support for Utf8View"`. In terms of scope of work between between #7434 and #6965, don't see a major overlap between what's intended. Therefore, I think the code conflict between the two should be minimal, and fairly trivial to resolve. > Thank you @kumarlokesh -- this looks really nice > > To make reviewing this PR easier, is there any way you can break it into multiple smaller PRs? Specifically, > > 1. Improve the comments > 2. Support for Map > 3. Support for Utf8View? > > Also it is not clear to me how much of a conflict this PR would have with: > > * [Avro codec enhancements #6965](https://github.com/apache/arrow-rs/pull/6965) -- 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
