-
Notifications
You must be signed in to change notification settings - Fork 78
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
fix connection not release bug #353
Changes from 5 commits
3a80718
517be56
f6ddf1d
2923348
dd1c11b
40de17e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -171,6 +171,8 @@ def authenticate(self, user_name, password): | |
try: | ||
resp = self._connection.authenticate(user_name, password) | ||
if resp.error_code != ErrorCode.SUCCEEDED: | ||
self._connection.is_used = False | ||
self._connection = None | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Authenticate failed does not mean the connection is broken, Why Let's see the connection pool how to call
the auth operation is undependent with connection, so auth failed does not Influence the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I refer to the code under There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done as agreed. |
||
raise AuthFailedException(resp.error_msg) | ||
return AuthResult( | ||
resp.session_id, resp.time_zone_offset_seconds, resp.time_zone_name | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the transport is not closed, prefer
self.close()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it necessary to close the connection here? I tend to release the connection back to the connection pool and wait for the next authenticate