Skip to content

revert: "chore: revert telemetry stabilization (#819)" - #828

Merged
hln33 merged 1 commit into
mainfrom
revert/819-restore-telemetry-stabilization
Oct 26, 2026
Merged

hln33 merged 1 commit into
mainfrom
revert/819-restore-telemetry-stabilization

Conversation

@hln33

@hln33 hln33 commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Reverts #819 (commit e512588a8273c225e931aa3cc6e86b5b0ceb4c22). This restores the telemetry-exporter state from #809.

Description

This PR removes the @Experimental annotation and its import from 27 public types in the telemetry-exporter module. It also removes the "(preview)" heading and the Preview API note from telemetry-exporter/README.md.

The diff is the exact inverse of the #819 squash commit. All 28 files are under telemetry-exporter/. Runtime implementations, API signatures, and Maven versions are unchanged. No commit merged after #819 touches telemetry-exporter/, so the revert applied without conflicts.

No package publication, deployment, merge, or auto-merge is requested.

Demo/Screenshots

Not applicable. This is a revert of stability markers and documentation.

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Validated locally with Maven on top of main at 541ea406.

  • Confirmed the committed diff is byte-identical to the inverse of e512588a (28 files, +1/-69).
  • mvn -B -pl telemetry-exporter -am clean verify: BUILD SUCCESS. 246 tests ran in telemetry-exporter; 0 failures.
  • mvn -B spotless:check: passed.

Unit Tests

Existing unit tests are used. No new test is added.

Integration Tests

Existing tests are used. Cloud tests are not enabled.

Examples

No new example. Runtime implementations and API signatures are unchanged.

@hln33
hln33 requested a review from a team October 26, 2026 19:33
@hln33
hln33 deployed to ai-pr-review-runtime October 26, 2026 19:34 — with GitHub Actions Active
@hln33
hln33 deployed to ai-pr-review-runtime October 26, 2026 19:35 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

Codex AI review

No actionable findings. Changes are limited to removing experimental annotations and preview documentation; runtime logic and API signatures remain unchanged. Tests were not run under the read-only review constraints.

Reviewed commit df27a1d77219dae7933bf20b9128371bf73b29a8. Workflow run

@hln33
hln33 merged commit 3543171 into main Oct 26, 2026
25 checks passed
@hln33
hln33 deleted the revert/819-restore-telemetry-stabilization branch October 26, 2026 19:43

This branch was successfully deployed

1 active deployment
ai-pr-review-runtime — df27a1d7 Deployed Oct 26, 2026 by hln33 via ai-pr-review / Codex review / Generate Codex review #2698
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