Skip to content

Fix signal name in OTLP gRPC logs package javadoc - #8737

Merged
jack-berg merged 1 commit into
open-telemetry:mainfrom
thswlsqls:docs/fix-otlp-grpc-logs-package-info-signal
Aug 20, 2026
Merged

Fix signal name in OTLP gRPC logs package javadoc#8737
jack-berg merged 1 commit into
open-telemetry:mainfrom
thswlsqls:docs/fix-otlp-grpc-logs-package-info-signal

Conversation

@thswlsqls

Copy link
Copy Markdown
Contributor

Description

  • The package javadoc of io.opentelemetry.exporter.otlp.logs says "sends span data", but the package exports logs.
  • Changes that one word to "log data", matching the sibling io.opentelemetry.exporter.otlp.http.logs package.
  • The wrong word dates back to Add OTLP logs exporter #3670, which created the logs package by copying the trace package.
  • Related: Fix copy-paste leftovers in public API javadoc #8693 fixed the same kind of copy-paste leftover in public API javadoc.

Testing done

  • Javadoc-only, test-exempt: no test added.
  • ./gradlew :exporters:otlp:all:check — BUILD SUCCESSFUL, 1047 tests passed (13 skipped), 0 failures.
  • No signature change, so docs/apidiffs/current_vs_latest/ is unchanged.

@thswlsqls
thswlsqls marked this pull request as ready for review August 20, 2026 06:18
@thswlsqls
thswlsqls requested a review from a team as a code owner August 20, 2026 06:18
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Aug 20, 2026

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-08-20 14:26 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.27%. Comparing base (0ae8f33) to head (a00c24d).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8737   +/-   ##
=========================================
  Coverage     91.27%   91.27%           
  Complexity    10471    10471           
=========================================
  Files          1006     1006           
  Lines         28276    28276           
  Branches       3568     3568           
=========================================
  Hits          25808    25808           
  Misses         1675     1675           
  Partials        793      793           

☔ 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.

@jack-berg
jack-berg merged commit 76bfd94 into open-telemetry:main Aug 20, 2026
33 checks passed
@otelbot

otelbot Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution @thswlsqls! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

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