qstommyshu commented on PR #15444: URL: https://github.com/apache/datafusion/pull/15444#issuecomment-2759953861
> In general I suggest we get the basic easy to port tests done and then work on the others as a follow on PR Got it, thanks @alamb and @blaginin for the help. I think I've finished migrating all the "easy" portion now, please review this PR. The only thing left are the non-trivial cases in `roundtrip_logical_plan.rs`. I will create another PR for that as suggested. -- 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
