2010YOUY01 commented on PR #15525: URL: https://github.com/apache/datafusion/pull/15525#issuecomment-2771286594
> Thanks @2010YOUY01 -- I think it makes sense to me to remove unused code > > I think this functionality would be used to implement `ORDER BY LIMIT <VERY LARGE NUMBER>` -- aka where the TopK rows wouldn't fit into the memory budget > > However given the lack of test coverage it is clear this code is not used anymore > > Perhaps you can also file a ticket to track supporting "topK" queries with limited memory when the K is "large"? Thank you. In order to support external `TopK`, I wrote up the idea in https://github.com/apache/datafusion/issues/15538 -- 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
