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

Draft: add psycopg3 pool #298

Merged
merged 18 commits into from
Jan 8, 2025
Merged

Draft: add psycopg3 pool #298

merged 18 commits into from
Jan 8, 2025

Conversation

akerlay
Copy link
Contributor

@akerlay akerlay commented Sep 30, 2024

No description provided.

Copy link
Collaborator

@kalombos kalombos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have a look at the PR. Looks great! I'm going to look into it in more detail a little later.

if self.pool is None:
await self.connect()
assert self.pool is not None, "Pool is not connected"
return await self.pool.getconn()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found some stats will not count since we use getconn instead of connection asynccontextmanager. https://github.com/psycopg/psycopg/blob/master/psycopg_pool/psycopg_pool/pool_async.py#L198

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I created an issue #300 No need to fix in the PR

@kalombos kalombos mentioned this pull request Nov 9, 2024
@kalombos kalombos merged commit 922068c into 05bit:master Jan 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants