feat(website): restructure the benchmark page into clearer sections#281
Merged
Conversation
Reorganise /benchmarks under Performance results, Code comparison, Inspect each workload, and Methodology and reproduction, without changing any measured data. - New opening copy and a workload-coverage line; the first stat card reads "fastest framework above JDBC". - Reword the network-overhead sentence and give the JDBC round-trip figure as a 155-172 us range across sessions; qualify the LOC comparison. - Add a Scope and limitations box and a closing call to action. - Each workload shows Storm by default, the other six libraries a selector away and the SQL a keyboard-operable Show SQL toggle away.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Restructures
/benchmarksfor readability. No benchmark data, numbers, rankings, error margins, versions or benchmark source are changed — every value renders from the sameWORKLOADSconstant as before.Sections
The page is reorganised under four clear headings: Performance results, Code comparison, Inspect each workload, and Methodology and reproduction. No content was removed.
Copy
/quickstart, View benchmark source → the benchmark repo).Inspect each workload
Each workload shows Storm's implementation by default; the other six libraries are one
<select>away and the SQL one "Show SQL" toggle away, both in the editor's own title bar (no extra panels or toggle links). The Show SQL toggle is keyboard-operable (role="button",tabindex,aria-expanded, Enter/Space) — the only accessibility gap in the editor chrome.Verification
npm run buildsucceeds. Broken-link warnings in the build are pre-existing/docs/*versioned-docs anchors, unrelated to this change.