rdblue commented on pull request #1487: URL: https://github.com/apache/iceberg/pull/1487#issuecomment-704412750
Thanks for the explanation. My general preference is for tests with a smaller scope to help identify the part that is going wrong, rather than larger end-to-end tests that could fail in lots of places. But `TestFilteredScan` is already exercising more than just the filter pushdown during planning, so both are similar in scope. I'll merge this and defer to you about how we should maintain these two. Thanks for taking the time to write these. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
