zhuqi-lucas commented on PR #15380: URL: https://github.com/apache/datafusion/pull/15380#issuecomment-2812444262
> > Tested latest PR, it seems regression again for current code... > > Hmm... could it be because of the increased threshold? I think if we're strictly faster and keep the same heuristic we shouldn't have regressions? I did not change the threshold, i found it's not stable for the result, but it's will 100% improve for the sort by one column case. But for mulit column sorting, it will not increase a lot from current code. -- 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
