veluca93 commented on PR #6021: URL: https://github.com/apache/arrow-rs/pull/6021#issuecomment-2215448903
> Thanks @veluca93 -- I looked at some of these changes -- some places definitely look like there could problems, though I am not quite sure > > I wonder if you would be willing to break this PR into smaller independent PRs for review > > for example, > > 1. the changes to parquet might go in one PR > 2. Changes for list/map arrays in another > 3. changes for dictionary array in another > > This would both be easier to review and let us merge the parts that get consensus while we work through the others > > cc @XiangpengHao and @tustvold @jhorstmann for your comments Happy to split the PR up! -- 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]
