alamb commented on PR #17357: URL: https://github.com/apache/datafusion/pull/17357#issuecomment-3261789341
> > Thanks @chenkovsky and @nuno-faria -- I think this PR is quite good and probably can be merged. My only potential concern is that we may mess up comet. Let's see if we get any more comments > > Comet was actually just thinking about coalesce: > > [apache/datafusion-comet#2270](https://github.com/apache/datafusion-comet/pull/2270) > > Maybe @coderfender has some thoughts about this? It seems like this PR does the same thing described in the comet PR - https://github.com/apache/datafusion-comet/pull/2270 Thus it seems like a good thing to merge Let's wait a while to see if anyone else has comments, otherwise I'll plan to merge this PR -- 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
