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

Simplify the database pool settings #210

Merged
merged 2 commits into from
Sep 19, 2022
Merged

Conversation

tchoutri
Copy link
Contributor

@tchoutri tchoutri commented Sep 19, 2022

Proposed changes

This PR removes the "connection" and "sub-pools" distinction, in favour of a single "connections" configuration variable. By default it is set to 100, which corresponds to the previous value of 10 × 10.

Contributor checklist

  • My PR is related to <insert ticket number>
  • I have read and understood the CONTRIBUTING guide
  • I have inserted my change and a link to this PR in the CHANGELOG

This commit removes the "connection" and "sub-pools" distinction,
in favour of a single "connections" configuration variable.
By default it is set to 100, which corresponds to the
previous value of 10 × 10.
@tchoutri tchoutri added the squash+merge me Tell Mergify bot to squash and merge the PR label Sep 19, 2022
@mergify mergify bot merged commit fb87a2d into development Sep 19, 2022
tchoutri added a commit that referenced this pull request Sep 20, 2022
* Simplify the database pool settings

This commit removes the "connection" and "sub-pools" distinction,
in favour of a single "connections" configuration variable.
By default it is set to 100, which corresponds to the
previous value of 10 × 10.

* Add changelog entry
@tchoutri tchoutri deleted the change-pool-settings branch March 21, 2023 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squash+merge me Tell Mergify bot to squash and merge the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant