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.

Sign up for GitHub

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

Jump to bottom

set_session() uses default_transaction_* which doesn't play well with connection polling #129

Open
nicoe opened this issue Feb 28, 2022 · 0 comments
Open

set_session() uses default_transaction_* which doesn't play well with connection polling #129

nicoe opened this issue Feb 28, 2022 · 0 comments

Comments

@nicoe
Copy link

nicoe commented Feb 28, 2022

I was wondering if I could replace my usage of set_isolation_level by set_session in some of my code and a comment reminded me why we didn't do it earlier : it does not play well with connection polling.

Apparently psycopg2 already fixed the bug: psycopg/psycopg2#503
Do you think the same strategy could work with psycopg2cffi (ie: construct the begin statement that should be sent to the database)?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nicoe