jonathanc-n commented on PR #16660: URL: https://github.com/apache/datafusion/pull/16660#issuecomment-3203939654
@comphead @2010YOUY01 I have added incremental processing, performance cost docs, and diagrams in the tests to verify correctness easier. I will make the incremental processing batch limit to be configurable as a runtime config once planner is implemented. I will do the following as follow-up PRs + put it in an EPIC issue (this pull request was getting quite large and cluttered, probably better to merge this for now if it looks ready): - Fuzz test - Planner + AND predicate handling -> SLT tests (i'd like to do the AND expression + planner together as right now Datafusion doesn't have too good of a method to separate IE predicates and equal predicates) - Mark joins - Serialization Thank you very much so far for the reviews, really appreciate it. -- 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