Skip to content

blog: clustering part 1 - #124

Merged
spetz merged 2 commits into
mainfrom
clustering_blog_part_1
Sep 28, 2026
Merged

spetz merged 2 commits into
mainfrom
clustering_blog_part_1

Conversation

@numinnex

Copy link
Copy Markdown
Contributor

No description provided.

@justinmclean

Copy link
Copy Markdown
Member

Great post. I checked every latency in the tables against the linked reports and they all match, and the derived unthrottled throughputs recompute correctly from the per-producer data.

A few suggested minor changes:

  • Version label: use 0.9.0 instead of 0.9.0-rc.1 on line 60 and in the table rows. It's the same commit, and the 0.9.0 release post quotes these same numbers as 0.9.0, so one name for both is clearer.
  • Line 54: v0.9.0-edge.2 was an edge build, not a release. At the ASF a release is what the PMC votes on, so "the first edge build with exactly one server in it" is safer.
  • Line 82: line 60 says anything under 1.7× is noise, but this paragraph reads the P99 change (1.36×) as a real cost of the rewrite.
  • Line 134: the cold-read throughputs (2,713 and 2,710 MB/s) are derived, so they don't match the 2,812 and 2,833 MB/s that their linked reports show. "Derived as above" would save readers some confusion.
  • Line 64: 1,134 MB/s is the cluster's total, not a per-node figure, so "Throughput" fits the header better than "Throughput/node".

@hubcio hubcio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checked against server-0.9.0 (71f29618b), benchmark numbers taken as given. links, the linked table values, the code snippet and the git numbers all check out - the comments are on the parts that do not.

Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
Comment thread content/blog/vsr-clustering-part-1.mdx Outdated
hubcio
hubcio previously approved these changes Sep 28, 2026
@spetz
spetz merged commit 0c8cf01 into main Sep 28, 2026
2 checks passed
@spetz
spetz deleted the clustering_blog_part_1 branch September 28, 2026 14:39
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.

4 participants