berkaysynnada commented on PR #15993: URL: https://github.com/apache/datafusion/pull/15993#issuecomment-2869869805
@irenjj I've set the default value of this config to "false", and auto-complete the slt tests. This is one of the changes: <img width="734" alt="image" src="https://github.com/user-attachments/assets/a277abe6-c00f-4838-bbbd-4b5270adcd2c"; /> as I said, even if the logical planner doesn't remove the Sort, it's still eliminated in the physical planning stage. I also thought it's so because of the count(), but if I replace it by another aggregate fn, a SortExec still doesn't emerge in the physical plan (I also consider subquery brings unique results, so remove the inner group by also to be sure). TLDR, we cannot disable subquery sort elimination by disabling it from the logical world only. -- 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
