Skip to content

ref(opentelemetry): Remove deprecated OpenTelemetry classes - #6243

Open
adinauer wants to merge 5 commits into
9.x.xfrom
ref/remove-legacy-opentelemetry-classes-v9
Open

adinauer wants to merge 5 commits into
9.x.xfrom
ref/remove-legacy-opentelemetry-classes-v9

Conversation

@adinauer

@adinauer adinauer commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

📜 Description

Remove the deprecated SentrySpanProcessor, SentryPropagator, OpenTelemetryLinkErrorEventProcessor, and internal TraceData classes from sentry-opentelemetry-core. Remove the now-unused SentrySpanStorage, their legacy-only tests, and update the OpenTelemetry documentation and API dumps.

💡 Motivation and Context

These classes were deprecated before the Java SDK 9 release. OtelSentrySpanProcessor and OtelSentryPropagator are the supported replacements, while OpenTelemetryLinkErrorEventProcessor and TraceData are no longer needed. The current integration uses SentryWeakSpanStorage instead of SentrySpanStorage.

💚 How did you test it?

  • ./gradlew spotlessApply apiDump
  • ./gradlew ':sentry-opentelemetry:sentry-opentelemetry-core:check'
  • ./gradlew ':sentry:check'

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

None.

adinauer and others added 2 commits October 12, 2026 06:34
Remove the legacy span processor, propagator, error event processor, and trace
data classes for the 9.0 release. Keep the Otel-prefixed integration classes as
the supported implementations.

Fixes #6013

Co-Authored-By: Claude <noreply@anthropic.com>
@sentry

sentry Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.59.0 (1) release

⚙️ sentry-android Build Distribution Settings

@lbloder lbloder left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good!

With these legacy classes removed, there might be an opportunity for further cleanups:

  • io.sentry.SentrySpanStorage seems to be unused now, so we could delete that class as well.
  • I Couldn't find any usage of io.sentry.Instrumenter.OTEL anymore (except for tests). So we could potentially remove that. Then only Instrumenter.SENTRY stays which means we could get ride of the option and the enum altogether.

adinauer and others added 2 commits October 13, 2026 16:55
Remove the unused legacy OpenTelemetry span storage. The current integration uses
SentryWeakSpanStorage from sentry-opentelemetry-bootstrap instead.

Co-Authored-By: Claude <noreply@anthropic.com>
@adinauer

Copy link
Copy Markdown
Member Author

I Couldn't find any usage of io.sentry.Instrumenter.OTEL anymore (except for tests). So we could potentially remove that. Then only Instrumenter.SENTRY stays which means we could get ride of the option and the enum altogether.

I'll remove instrumenter in a separate PR to not bloat this one.

@adinauer
adinauer marked this pull request as ready for review October 13, 2026 15:05
@adinauer

Copy link
Copy Markdown
Member Author

Follow-up: the deprecated Instrumenter option and related code are removed separately in #6252.

This branch has not been deployed

No deployments
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