Skip to content

Rework UI#838

Open
rotmanjanez wants to merge 4 commits intoClickHouse:mainfrom
rotmanjanez:rework-ui
Open

Rework UI#838
rotmanjanez wants to merge 4 commits intoClickHouse:mainfrom
rotmanjanez:rework-ui

Conversation

@rotmanjanez
Copy link
Copy Markdown

I regularly look at the ClickBench website, and slowly, the selectors become more and more convoluted, and viewing the actual results becomes more annoying. This is an attempt to fix the UI experience of the online benchmark results websites.

Main Changes:

  1. Move Combined Benchmark ranking to the top. It is a bit odd going to a benchmark site and only seeing a bunch of buttons and no results. Moving the table to the top instantly shows benchmark results to first visitors and long-time viewers. Making the combined table scrollable should also improve viewing the site on mobile devices (though still far away from mobile-friendly).
  2. Update System selectors: There are eleven versions of ClickHouse and nine versions of DuckDB in the system selector filter. If I am not interested in DuckDB for whatever reason, I’d like to disable/enable all of them. This change introduces a new Database selector that buckets the different variations into more easily digestible selectors. The full systems selector is still in a dropdown.
  3. Change default metric back to hot. All other subpages in the repo also have hot as their default. The HTML comparing different ClickHouse versions does not even implement the combined metric. It does not seem very important to technical persons, who are likely the main visitors of the page. The first thing I do when I visit clickbench is to change to hot run only, and I know that I am not the only one doing this. I would be interested if anyone that actually uses this site on a regular basis looks at the combined metric and why.

I tried to make old links with hashes still work, and from my limited testing, they still work as intended.

I hope you and others like the direction of this PR.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 29, 2026

CLA assistant check
All committers have signed the CLA.

@alexey-milovidov alexey-milovidov marked this pull request as draft May 2, 2026 00:06
@rotmanjanez rotmanjanez marked this pull request as ready for review May 2, 2026 11:19
this also seems like a more interresting metric, as the other HTMLs did not seem to bother adding the combined few. This suggests that the hot-metric is mostly used and should be the default also here
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants