Dandandan commented on PR #15697: URL: https://github.com/apache/datafusion/pull/15697#issuecomment-2816249151
> FYI @Dandandan although very rough I put up a draft of filter pushdown in #15770. > > The interaction with this PR is something to think about. In particular it’d be nice if TopK could know if it’s filter got pushed down as exact or not, or if a FilterExec got created with that filter, so that it can avoid re-filtering Yeah would be good (I think it could follow when pushding down the filter into the scan is completed). Probably the effect would be small, as the filter itself seems not very expensive and if topk is pushed down, the number of rows coming into topk is also itself significantly reduced. -- 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
