Dandandan commented on PR #15380: URL: https://github.com/apache/datafusion/pull/15380#issuecomment-2888419860
So it seems re > I am confused why my benchmark for local Mac no regression for sort-tpch Q3, but the generated benchmark for linux we can reproduce the regression. I was wondering, maybe we only add the "concat-array-only" optimization and leave the rest for what it is? So don't change the `if self.reservation.size() < self.sort_in_place_threshold_bytes` to use different heuristics and keep `sort_in_place_threshold_bytes` to `1024 * 1024`. It might be of smaler significance, but it is more likely to have no or smaller regressions (and we can follow-up with better heuristics that work accros different machines). -- 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
