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

Clarify PgBouncer Compatibility #1338

Merged
merged 1 commit into from May 18, 2024

Conversation

isaac
Copy link
Contributor

@isaac isaac commented Apr 25, 2024

The compatibility story regarding PgBouncer in the README confused me so this is my attempt at clarification.

I would also like to clarify what the implications of the following paragraph are, but I am still unclear on this.

GoodJob is not compatible with PgBouncer in transaction mode, but is compatible with PgBouncer's connection mode. GoodJob uses connection-based advisory locks and LISTEN/NOTIFY, both of which require full database connections.

For example: does this mean that the GoodJob dashboard will work fine with PgBouncer, but actually running jobs will not work?

Thanks again for all your work on this project!

@isaac isaac force-pushed the clarify-pgbouncer-compatibility branch from 248a0c3 to 30fb4a7 Compare April 25, 2024 19:08
@bensheldon
Copy link
Owner

Thanks for improving the documentation! 🙇🏻

does this mean that the GoodJob dashboard will work fine with PgBouncer, but actually running jobs will not work?

The Dashboard uses the same Active Record models as the jobs / executors, so it will/must use the same database connections which won't work with PgBouncer. The Dashboard also can perform some job operations that require taking advisory locks, and those wouldn't work correctly with PgBouncer.

@bensheldon bensheldon added the documentation Improvements or additions to documentation label May 18, 2024
@bensheldon bensheldon merged commit 02f016c into bensheldon:main May 18, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants