Conversation
Splits the single benchmarks doc into a landing page plus separate Local File Source and Remote Windows Event Log (RWEL) pages, since RWEL's WinRM-bound throughput ceiling produces a different performance profile than local file sources. Fixes two Remote Windows Event Log source docs that linked to the old combined page path.
kimsauce
left a comment
There was a problem hiding this comment.
Structure, sidebar wiring, and internal links all check out, and CI is green (spellcheck, build checks). Left three inline notes on remote-windows-event-log-benchmarks.md: an inconsistent heading level, a sentence running into the previous bullet from a missing blank line, and a missing id field in the frontmatter. None of that blocks merging.
Generated by Claude Code
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
…-file-source-benchmarks.md Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
…e-windows-event-log-benchmarks.md Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
…-file-source-benchmarks.md Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Fixes heading level and missing bullet in remote-windows-event-log-benchmarks.md, adds an IC vs OTel Performance Benchmarks card to the Send Data cards list, and removes a duplicate id key in local-file-source-benchmarks.md left over from applying a suggested edit.
kimsauce
left a comment
There was a problem hiding this comment.
Re-reviewed after the latest push. All prior findings are fixed: the heading level on remote-windows-event-log-benchmarks.md is now consistent (### Test conditions), the missing bullet before "Neither collector exceeded the 190% safety cutoff..." is in, id fields are set on both new/renamed pages, the landing page now uses slug per the suggested fix, and a card for the new landing page was added to docs/send-data/index.md. CI is green (spellcheck, build checks, merge window). Nothing further from me.
Generated by Claude Code
Purpose of this pull request
This pull request splits the single "Installed Collector vs OpenTelemetry Collector Performance Benchmarks" doc into a landing page plus two collection-method-specific pages, since Remote Windows Event Log (RWEL) collection has a fundamentally different performance profile than local file sources (WinRM-bound throughput ceiling vs. a linear ramp).
ic-vs-ot-collector-performance-benchmarks/index.md, with card links to both sub-pageslocal-file-source-benchmarks.md: existing local file source benchmark content (renamed from the original single-file doc)remote-windows-event-log-benchmarks.md: new RWEL-specific benchmark data, with its own recommendation, test environment, and results tablessidebars.tsto a category linking to the new index and its two childrenSelect the type of change
Ticket (if applicable)
https://sumologic.atlassian.net/browse/DOCS-1875