Skip to content

Introduce SPID connection property - #660

Merged
cheenamalhotra merged 5 commits into
dotnet:masterfrom
cheenamalhotra:spid
Jul 23, 2020
Merged

Introduce SPID connection property#660
cheenamalhotra merged 5 commits into
dotnet:masterfrom
cheenamalhotra:spid

Conversation

@cheenamalhotra

@cheenamalhotracheenamalhotra commented Jul 21, 2020

Copy link
Copy Markdown
Member

Addresses #657

Introduces new SqlConnection's readonly property:
ServerProcessId: The server process Id (SPID) of the active connection.

cc @shueybubbles

shueybubbles
shueybubbles suggested changes Jul 21, 2020
View reviewed changes
Comment threadsrc/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectivityTests/ConnectivityTest.cs Outdated
JRahnama
JRahnama reviewed Jul 21, 2020
View reviewed changes
Comment threadsrc/Microsoft.Data.SqlClient/netfx/src/Resources/Strings.resx Outdated
JRahnama
JRahnama reviewed Jul 21, 2020
View reviewed changes
Comment threadsrc/Microsoft.Data.SqlClient/netfx/src/Microsoft.Data.SqlClient.csproj Outdated
David-Engel
David-Engel reviewed Jul 21, 2020
View reviewed changes
Comment threaddoc/snippets/Microsoft.Data.SqlClient/SqlConnection.xml Outdated
Comment threadsrc/Microsoft.Data.SqlClient/netfx/src/Microsoft.Data.SqlClient.csproj Outdated
@cheenamalhotracheenamalhotra linked an issue Jul 21, 2020 that may be closed by this pull request
Expose ProcessId (SPID) as a public property on SqlConnection #657
Closed
@cheenamalhotracheenamalhotra added this to the 2.1.0-preview1 milestone Jul 22, 2020
karinazhou
karinazhou reviewed Jul 23, 2020
View reviewed changes
Comment threadsrc/Microsoft.Data.SqlClient/netfx/src/Resources/Strings.resx
karinazhou
karinazhou reviewed Jul 23, 2020
View reviewed changes
Comment threadsrc/Microsoft.Data.SqlClient/netfx/src/Resources/Strings.Designer.cs
@cheenamalhotra
cheenamalhotra merged commit 580b172 into dotnet:masterJul 23, 2020
@cheenamalhotracheenamalhotra added the Public API 🆕 Issues/PRs that introduce new APIs to the driver. label Jul 23, 2020
@dependabotdependabotBot mentioned this pull request Jul 23, 2025
Open
@dependabotdependabotBot mentioned this pull request Dec 30, 2025
Merged
@dependabotdependabotBot mentioned this pull request Feb 26, 2026
Closed
@cheenamalhotra
cheenamalhotra deleted the spid branch April 1, 2026 16:22
This was referenced May 22, 2026
Open
Open
@dependabotdependabotBot mentioned this pull request Jun 1, 2026
Open
This was referenced Aug 11, 2026
Open
Open
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@David-EngelDavid-EngelDavid-Engel approved these changes
+3 more reviewers
@shueybubblesshueybubblesshueybubbles approved these changes
@karinazhoukarinazhoukarinazhou approved these changes
@JRahnamaJRahnamaJRahnama approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Public API 🆕Issues/PRs that introduce new APIs to the driver.

Projects

None yet

Milestone

2.1.0-preview1

Development

Successfully merging this pull request may close these issues.

Expose ProcessId (SPID) as a public property on SqlConnection

5 participants

@cheenamalhotra@shueybubbles@David-Engel@karinazhou@JRahnama