Skip to content

docs(logs): polish design doc and fill perf numbers#3566

Open
cijothomas wants to merge 3 commits into
open-telemetry:mainfrom
cijothomas:cijothomas/docs-logs-polish
Open

docs(logs): polish design doc and fill perf numbers#3566
cijothomas wants to merge 3 commits into
open-telemetry:mainfrom
cijothomas:cijothomas/docs-logs-polish

Conversation

@cijothomas

Copy link
Copy Markdown
Member

Polish pass over docs/design/logs.md:

  • Minor wording/typo fixes in the API and LogProcessor sections.
  • Fill in Perf test - benchmarks and Perf test - stress test with numbers and links to the actual benchmarks (opentelemetry-appender-tracing/benches/logs.rs) and the stress harness (stress/src/logs.rs), and add a short framing paragraph to the Performance section.
  • Internal logs section: replace the two // TODO placeholders. The severity-selection TODO is dropped (out of scope for a Logs-signal design doc). The "OTLP can solve this" TODO is replaced with a short paragraph noting the open suppression gap for hyper/tonic and linking the tracking issue (Fix telemetry-induced-telemetry in OTLP Exporter #2877).

No code or behavior changes.

@cijothomas
cijothomas requested a review from a team as a code owner June 23, 2026 17:21
@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.2%. Comparing base (0e78170) to head (a9c2c8c).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #3566   +/-   ##
=====================================
  Coverage   83.2%   83.2%           
=====================================
  Files        130     130           
  Lines      28246   28246           
=====================================
  Hits       23506   23506           
  Misses      4740    4740           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread docs/design/logs.md Outdated
@github-actions

Copy link
Copy Markdown

Thank you for your contribution! This PR has been automatically marked as stale because it has not had activity in the last 14 days. This may be due to a delay in review on our side or awaiting a response from you; either is fine, and we appreciate your patience.

It will be closed in 14 days if no further activity occurs. Pushing a new commit or leaving a comment will remove the stale label and keep the PR open.

@github-actions github-actions Bot added the Stale label Jul 17, 2026
@cijothomas

Copy link
Copy Markdown
Member Author

@utpilla @lalitb PTAL.

@github-actions github-actions Bot removed the Stale label Jul 23, 2026
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.

2 participants