Skip to content

Rework UI#838

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

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
@rschu1ze
Copy link
Copy Markdown
Member

rschu1ze commented May 4, 2026

@rotmanjanez I generally like the direction, thanks. Some comments:

  1. Move Combined Benchmark ranking to the top.

Makes sense.

Making the combined table scrollable should also improve viewing the site on mobile devices (though still far away from mobile-friendly).

Not a blocker: The (minor) problem here is that if only few databases are selected, the graph is still padded.

grafik

Can the graph height be auto-adjusted or be made configurable via some click-on-the-border-and-drag feature?

  1. Update System selectors.

This one is a bit more controversial. My main quirk is that the "Additional filters" button is too hard to spot, IMHO.

grafik

Most users will need to click it open because extra filtering is a necessity. So we can show the other filters always (as before), i.e. no filter button.

  1. Change default metric back to hot.

This reverts #616 and it is the most controversial change :-) I'd say real workloads are a mix of cold and hot runtimes anyways, so "combined" seems a good default choice. Viewers can switch to hot or cold if they like to. Could I ask you to revert this change?

Thanks for the PR (you are clearly much better with HTML and JS than me ;-) )

@alexey-milovidov alexey-milovidov marked this pull request as draft May 5, 2026 18:49
@alexey-milovidov alexey-milovidov self-assigned this May 5, 2026
@rschu1ze rschu1ze self-assigned this May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants