adriangb commented on PR #15770: URL: https://github.com/apache/datafusion/pull/15770#issuecomment-2957654244
@berkaysynnada I think I made some progress. I was able to get the TopK pushdown tests passing by passing around the filter in the `EnforceSorting` rule as is already done for the `fetch` parameter. I struggled to wrap my head around this new to me bit of code and I mostly was taking stabs in the dark to try to understand it / get it to work, so as tests reflect I fear I may have broken some invariant. Could you take a look at 16fd477 and if you think that's a ballpark reasonable approach we schedule a call to try to fix it up together? -- 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
