andygrove commented on PR #1206: URL: https://github.com/apache/datafusion-comet/pull/1206#issuecomment-2569642133
@rluvaton I think that this looks great but it is challenging to really review the PR with so many changes.  What do you think about breaking this down into some smaller PRs that are easier to review? For example, maybe the first PR just moves string expressions. -- 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