Skip to content

chore: Deprecate db.query op in favor of db - #760

Draft
sentry-junior[bot] wants to merge 1 commit into
mainfrom
ls/deprecate-db-query-op
Draft

sentry-junior[bot] wants to merge 1 commit into
mainfrom
ls/deprecate-db-query-op

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

SDKs currently use a mix of db and db.query for database spans, and the conventions don't say when to use which. db is the more widely used of the two, so this deprecates db.query and points it at db. The details of the operation belong in the db.* span attributes.

  • db.query is marked deprecated, with db as its replacement.
  • db.redis now points at db too, because a replacement can't itself be deprecated.
  • db gets a short description, and the example on the ops docs page now uses db.

db.query stays in the name and description rules because older SDKs will keep sending it. Product queries that filter on span.category keep working, since the category comes from the first segment of the op. SDKs should switch in their next major version.

via Lukas Stracke.

--

View Session [Sentry]

Co-Authored-By: Lukas Stracke <lukas.stracke@sentry.io>
@vercel

vercel Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-conventions Ready Ready Preview Oct 30, 2026 11:08am UTC

Request Review

@github-actions

Copy link
Copy Markdown

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Internal Changes 🔧

  • Deprecate db.query op in favor of db by sentry-junior in #760
  • Deprecate gen_ai.handoff op by alexander-alderman-webb in #754

🤖 This preview updates automatically when you update the PR.

This branch was successfully deployed

1 active deployment
Preview — 8dbd3006 Deployed Oct 30, 2026 by vercel[bot]
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.

0 participants