gabotechs commented on PR #16029: URL: https://github.com/apache/datafusion/pull/16029#issuecomment-2903452433
> I also wonder about testing the test harness, but I think that may be unecessary given how often these tools are run My guess is that it's not too bad to not have tests in the new `Filter` struct, but not just my call though. What I'd do is if somebody runs into an issue, add the tests at that moment. -- 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
