justinmclean commented on PR #124: URL: https://github.com/apache/iggy-website/pull/124#issuecomment-5868959160
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". -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
