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

pgbouncer: Update pool size calculation algorithm #362

Merged
merged 5 commits into from
Jun 2, 2023

Conversation

vitabaks
Copy link
Owner

@vitabaks vitabaks commented Jun 1, 2023

Update pool size calculation algorithm and add comprehensive code comments.

This PR addresses an issue with the calculation of the total size of pools in the pgbouncer configuration. The previous implementation had a flaw that resulted in incorrect pool size calculations. In this PR, the algorithm for calculating the pool size has been fixed to calculate the sum of the pool sizes accurately.

Additionally, this PR introduces detailed comments throughout the codebase. The comments provide clear explanations of the logic and purpose of each task, making the code more maintainable and easier to understand for future contributors.

The changes in this PR enhance the reliability and readability of the pgbouncer configuration code, ensuring accurate pool size calculations and facilitating smoother maintenance in the future.

@vitabaks vitabaks self-assigned this Jun 1, 2023
Implement total pool size calculation enhancement for pgbouncer. Now it properly accounts for databases not defined in 'pgbouncer_pools' and optionally uses the 'pgbouncer_default_pool_size'. This provides a more accurate total pool size estimation.
@vitabaks vitabaks merged commit 81f52f7 into master Jun 2, 2023
17 checks passed
@vitabaks vitabaks deleted the improve-calculate-pool-size branch June 13, 2023 13:36
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.

None yet

1 participant