-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
redis's connection pool makes me wonder #2832
Labels
bug
Bug
Comments
petyaslavova
added a commit
that referenced
this issue
Feb 18, 2025
…wned by connection pool connection(#2832).
6 tasks
PR is created with a fix: #3514 |
petyaslavova
added a commit
that referenced
this issue
Feb 24, 2025
…wned by connection pool connection(#2832).
petyaslavova
added a commit
that referenced
this issue
Feb 24, 2025
…ed by connection pool connection(fixes issue #2832). (#3514) * Removing decreasing of created connections count when releasing not owned by connection pool connection(#2832). * Fixed another issue that was allowing adding connections to a pool owned by other pools. Adding unit tests. * Fixing a typo in a comment
Fixed with PR#3514 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/redis/redis-py/blob/master/redis/connection.py#L1515
Since you don't own this connection, it means that the program has been fork, and _created_connections has been initialized to 0, why subtract 1 again?Can someone explain it to me?,thanks
The text was updated successfully, but these errors were encountered: