alamb commented on issue #14115: URL: https://github.com/apache/datafusion/issues/14115#issuecomment-2589715949
BTW @Chen-Yuan-Lai I very much suggest doing this task as multiple smaller PRs if possible (e.g. make a PR to replace the use in Correlation, make another PR to replace a single other use) That will help us review the code more quickly and avoid blocking some of the changes if we hit some roadblock to one -- 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
