Rework UI#838
Conversation
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
|
@rotmanjanez I generally like the direction, thanks. Some comments:
Makes sense.
Not a blocker: The (minor) problem here is that if only few databases are selected, the graph is still padded.
Can the graph height be auto-adjusted or be made configurable via some click-on-the-border-and-drag feature?
This one is a bit more controversial. My main quirk is that the "Additional filters" button is too hard to spot, IMHO.
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.
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 ;-) ) |


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:
Databaseselector that buckets the different variations into more easily digestible selectors. The full systems selector is still in a dropdown.hotas their default. The HTML comparing different ClickHouse versions does not even implement thecombinedmetric. 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.