Skip to content
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.

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

Send client process ID to server on Java 9+ #1595

Merged
merged 2 commits into from Jun 11, 2021

Conversation

David-Engel
Copy link
Contributor

@David-Engel David-Engel commented Jun 3, 2021

Since we can easily do it in Java 9 and up, we should. If catching NoClassDefFoundError is objectionable, I'm open to other suggestions.

You can validate the behavior by querying sys.dm_exec_sessions when a connection is active:
select * from sys.dm_exec_sessions

@David-Engel David-Engel changed the title Send process ID to server on Java 9+ Send client process ID to server on Java 9+ Jun 3, 2021
@lilgreenbird lilgreenbird added this to In progress in MSSQL JDBC Jun 8, 2021
@lilgreenbird lilgreenbird moved this from In progress to Under Peer Review in MSSQL JDBC Jun 8, 2021
@lilgreenbird
Copy link
Member

/azp run Microsoft.mssql-jdbc.macOS

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@lilgreenbird
Copy link
Member

/azp run Microsoft.mssql-jdbc.macOS

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@lilgreenbird lilgreenbird merged commit 574b1e1 into microsoft:dev Jun 11, 2021
MSSQL JDBC automation moved this from Under Peer Review to Closed/Merged PRs Jun 11, 2021
@lilgreenbird lilgreenbird added this to the 9.3.2 milestone Jun 12, 2021
@cheenamalhotra
Copy link
Member

Just curious.. should this call be wrapped under JDBC 43 support check?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
MSSQL JDBC
  
Closed/Merged PRs
Development

Successfully merging this pull request may close these issues.

None yet

5 participants