Skip to content

Conversation

@XSAM
Copy link
Member

@XSAM XSAM commented Jun 12, 2025

Part of #2162, Split from #2236, as this part seems reached the consensus.

XSAM/otelsql#508 is the prototype to demonstrate how SET CONTEXT_INFO works. You can run the example on your local Docker environment.

@XSAM
Copy link
Member Author

XSAM commented Jun 12, 2025

This PR only has sql server part with SET CONTEXT. I don't get any objection about this part on the SIG meet, so I am thinking about merge this first to unblock part of #2236.

@trask @lmolkova PTAL. Thanks!

@arminru arminru changed the title Add dashbase context propagation via SET CONTEXT_INFO for SQL Server Add database context propagation via SET CONTEXT_INFO for SQL Server Jun 16, 2025
Copy link
Contributor

@jsuereth jsuereth left a comment

Choose a reason for hiding this comment

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

As discussed, this works for me now. The normative language is intentional, and we're moving specification of this sort into semantic conventions

XSAM and others added 3 commits June 25, 2025 23:54
Co-authored-by: Liudmila Molkova <[email protected]>
@github-project-automation github-project-automation bot moved this from Untriaged to Needs More Approval in Semantic Conventions Triage Jun 30, 2025
@joaopgrassi
Copy link
Member

CC @open-telemetry/semconv-db-approvers

@lmolkova lmolkova added this pull request to the merge queue Jul 1, 2025
Merged via the queue into open-telemetry:main with commit f0ed490 Jul 1, 2025
15 checks passed
@XSAM XSAM deleted the sql-server-set-context branch July 1, 2025 05:30
jherbage pushed a commit to jherbage/semantic-conventions that referenced this pull request Jul 1, 2025
spurplewang pushed a commit to spurplewang/semantic-conventions that referenced this pull request Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:db enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants