Skip to content

Conversation

@kemaldotearth
Copy link
Contributor

I have read the CONTRIBUTING.md file.

YES

What kind of change does this PR introduce?

This was missing for queries that still use pgbouncer as a role. I know it's deprecated, so I put the same message that we have on the blog post here.

Fixes this:
CleanShot 2025-09-23 at 21 30 28@2x

cc @jordienr who found the bug.

This was missing for queries that still use pgbouncer as a role.
@kemaldotearth kemaldotearth requested a review from a team as a code owner September 25, 2025 09:57
@vercel
Copy link

vercel bot commented Sep 25, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
studio-self-hosted Ready Ready Preview Comment Sep 29, 2025 8:01am
studio-staging Ready Ready Preview Comment Sep 29, 2025 8:01am
6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
cms Ignored Ignored Sep 29, 2025 8:01am
studio Ignored Ignored Sep 29, 2025 8:01am
design-system Skipped Skipped Sep 29, 2025 8:01am
docs Skipped Skipped Sep 29, 2025 8:01am
ui-library Skipped Skipped Sep 29, 2025 8:01am
zone-www-dot-com Skipped Skipped Sep 29, 2025 8:01am

@supabase
Copy link

supabase bot commented Sep 25, 2025

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@kemaldotearth kemaldotearth requested review from a team and jordienr September 25, 2025 09:57
@coveralls
Copy link

coveralls commented Sep 25, 2025

Coverage Status

coverage: 69.874%. remained the same
when pulling 3b14fb7 on feat/add-pgbouncer-description-query-perf
into 1973615 on master.

Copy link
Contributor

@kevcodez kevcodez left a comment

Choose a reason for hiding this comment

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

PgBouncer is not deprecated, quite the opposite - it's our recommended, dedicated pooler for any paid organization. Caveat is only available on paid plans + only supports IPv6

Supavisor is the shared pooler that should only be used on the Free Plan or if you're not in a ipv6 compatible environment

@kemaldotearth
Copy link
Contributor Author

PgBouncer is not deprecated, quite the opposite - it's our recommended, dedicated pooler for any paid organization. Caveat is only available on paid plans + only supports IPv6

Supavisor is the shared pooler that should only be used on the Free Plan or if you're not in a ipv6 compatible environment

@kevcodez cheers for clearing that up, what would you suggest writing there? Is there a more generic description of pgbouncer we can use? Maybe we can say something like:

PgBouncer is a lightweight connection pooler for PostgreSQL. Paid plans only.

@kemaldotearth kemaldotearth merged commit d1c986e into master Sep 29, 2025
19 of 20 checks passed
@kemaldotearth kemaldotearth deleted the feat/add-pgbouncer-description-query-perf branch September 29, 2025 08:31
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.

5 participants