adriangb commented on PR #15770: URL: https://github.com/apache/datafusion/pull/15770#issuecomment-2977763190
> I think it's great! I added one suggestion to minimize the changes to the explain output and maybe phrase the "pre" and "post" more as "normal" filter pushdown and "dynamic" filter pushdown? I think that might explain a bit more of what might be happening. > > Maybe @alamb can verify the (increased) performance improvements once more? Thanks! I have no strong opinions about the naming so I'll wait for Andrew to review to see if he has any thoughts otherwise I'll go with your suggestion 😄 -- 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