zhuqi-lucas commented on PR #17008: URL: https://github.com/apache/datafusion/pull/17008#issuecomment-3148947852
> > > Another potential idea is to simply switch to using the coalesce kernel directly (so we don't have to maintain a separate copy) > > > > > > * [Draft: Use upstream arrow `coalesce` kernel in DataFusion #16249](https://github.com/apache/datafusion/pull/16249) > > > > > > Thank you @alamb for the review! I agree that switching to the upstream coalesce kernel would be the ideal follow-up solution. Is there anything I can do to help push this forward? I'd be happy to contribute. > > Thanks @zhuqi-lucas -- as always super appreciated > > Maybe once we upgrade to DataFusion 56 you can revive #16249 ? I think it is basically ok, but would benefit from having someone else go through the code and double check. The existing failures I think were due to other changes made at the time, and I suspect (hope) it will "just work" Thank you @alamb , i will revive it after we upgrade to 56! -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
