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

Slight memory leak on new connection #1632

Closed
elFarto opened this issue Aug 4, 2021 · 1 comment
Closed

Slight memory leak on new connection #1632

elFarto opened this issue Aug 4, 2021 · 1 comment
Labels
Bug A bug in the driver. A high priority item that one can expect to be addressed quickly.
Projects

Comments

@elFarto
Copy link

elFarto commented Aug 4, 2021

I've been taking a very close look at the memory usage of a few of our applications, and I happen to come across the following 8kB String beginning com.microsoft.sqlserver.jdbc.SQLServerConnection:123456789101112.

A quick look through the source code revealed the issue:


loggingClassName is static, so you shouldn't be appending the current connectionId to it.

@lilgreenbird lilgreenbird added the Bug A bug in the driver. A high priority item that one can expect to be addressed quickly. label Aug 4, 2021
@lilgreenbird
Copy link
Member

hi @elFarto ,

Thank you for catch, we will get this fixed in the upcoming release.

@lilgreenbird lilgreenbird added this to Closed Issues in MSSQL JDBC Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug in the driver. A high priority item that one can expect to be addressed quickly.
Projects
MSSQL JDBC
  
Closed Issues
Development

No branches or pull requests

2 participants