Chen-Yuan-Lai commented on PR #16978: URL: https://github.com/apache/datafusion/pull/16978#issuecomment-3188982652
> Maybe we can try to port a few tests in one of the files to use iinsta to make sure we are good with the pattern before applying the pattern to the entire thing > > For example, a more explicit `Test` style struct might make the options being tested more efficient. > > I know this is a big ask, but I think getting these tests into better shape would really help the project Sure! I will try it soon -- 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