Lordworms commented on PR #6046: URL: https://github.com/apache/arrow-rs/pull/6046#issuecomment-2227104660
> > > > We could hold off merging this until that work is complete and I'll update this PR or I can do it as a separate PR. > > > > > > > > > I recommend we merge this PR, and then port/fix up the struct array statistics directly in arrow-rs [apache/datafusion#11289](https://github.com/apache/datafusion/pull/11289) (cc @Lordworms ) > > > My rationale is that we are more likely to find some struct array expertise in the arrow-rs repo than the datafusion repo. > > > > > > I agree, should I port the struct related function now? > > @Lordworms I don't think it should be part of this PR which is already huge. I think the easiest thing to do is to wait until this PR is merged (which should happen soon) then open a new PR in this repository with the struct changes. Got it, I'll wait for it to be merged, thanks for your work. -- 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]
