pmcgleenon commented on issue #13983: URL: https://github.com/apache/datafusion/issues/13983#issuecomment-2636181803
I've re-run this again following the [discussion on the ClickBench PR](https://github.com/ClickHouse/ClickBench/pull/301#discussion_r1941921542) Q28 needed an update, since the regex was not working correctly ``` Query 28: SELECT REGEXP_REPLACE(Referer, '^https?://(?:www\.)?([^/]+)/.*$', '\1') AS k, AVG(length(Referer)) AS l, COUNT(*) AS c, MIN(Referer) FROM hits WHERE Referer <> '' GROUP BY k HAVING COUNT(*) > 100000 ORDER BY l DESC LIMIT 25; ``` After that change, the Q28 performance for datafusion `44.0.0` has regressed compared to `43.0.0`. Based on the discussion over on ClickBench I think is expected since the Q28 results for all previous datafusion releases are invalid. <img width="1801" alt="Image" src="https://github.com/user-attachments/assets/f8a7a408-b276-4c62-bf4b-a53d83152207"; /> @Dandandan @alamb I've attached the full clickbench here if you want to take a look [clickbench-latest 2.html.zip](https://github.com/user-attachments/files/18669693/clickbench-latest.2.html.zip) -- 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
