Skip to content

Rename db getter methods to match stable semconv naming #15696

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
trask merged 5 commits into from
Jan 5, 2026
Merged

Rename db getter methods to match stable semconv naming #15696

trask merged 5 commits into from
Jan 5, 2026

Conversation

@trask
Copy link
Member

@trask trask commented Dec 19, 2025 •
edited
Loading

Part of #12608

@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Dec 19, 2025
trask added 2 commits December 19, 2025 10:20
@trask
fix
fba7a7d
@trask
Fix UnusedException warning in DbClientAttributesExtractor
6f30dfb 
Pass the caught exception as the cause when rethrowing UnsupportedOperationException
to avoid compilation error when -Werror is enabled.
@trask trask force-pushed the rename-db-getter-methods branch 2 times, most recently from f5847e0 to 0fc61c5 Compare December 19, 2025 19:48
@trask trask force-pushed the rename-db-getter-methods branch from 0fc61c5 to 4e72a14 Compare December 19, 2025 20:00
@trask trask marked this pull request as ready for review December 19, 2025 20:50
@trask trask requested a review from a team as a code owner December 19, 2025 20:50
@trask trask mentioned this pull request Dec 19, 2025
Open
20 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Dec 20, 2025

📋 Deprecation Notice

This PR has been labeled as a deprecation. Please ensure you provide the following information:

📝 Deprecation Details Required

Please add details to help users understand:

  • What is being deprecated and why
  • What should be used instead (if applicable)
  • Timeline for removal (if known)
  • Any migration guidance

📋 Checklist

  • Deprecation details added to the PR description
  • Deprecation is documented in the changelog entry for the next release
  • Consider adding deprecation warnings in code/documentation

Your deprecation notes will be included in the release notes to help users prepare for future changes.

This comment was automatically generated because the deprecation label was applied to this PR.

@trask trask merged commit 43afca6 into open-telemetry:main Jan 5, 2026
87 checks passed
@trask trask deleted the rename-db-getter-methods branch January 5, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jaydeluca jaydeluca jaydeluca approved these changes

Assignees

No one assigned

Labels

deprecation test native This label can be applied to PRs to trigger them to run native tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@trask @jaydeluca