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

docs: min_idle -> max_idle #96

Merged
merged 1 commit into from
Apr 30, 2024
Merged

docs: min_idle -> max_idle #96

merged 1 commit into from
Apr 30, 2024

Conversation

mnahkies
Copy link
Contributor

fixes a typo in the docs, ref https://github.com/importcjj/mobc/blob/main/src/config.rs#L12

(coincidentally I was looking to check if there is a concept of minimum pool size - eg: make sure that the pool always has at least X hot connections, so that an idle timeout can't result in an empty pool for bursty traffic patterns - though I guess setting max_idle rather than an idle timeout might be the recommendation here?)

fixes a typo in the docs, ref https://github.com/importcjj/mobc/blob/main/src/config.rs#L12

(coincidentally I was looking to check if there is a concept of minimum pool size - eg: make sure that the pool always has at least X hot connections, so that an idle timeout can't result in an empty pool for bursty traffic patterns - though I guess setting `max_idle` rather than an idle timeout might be the recommendation here?)
@garrensmith
Copy link
Collaborator

Yes correct. Set the max_idle to get what you want. Thanks for the fix.

@garrensmith garrensmith merged commit a9555f3 into importcjj:main Apr 30, 2024
1 check passed
@mnahkies mnahkies deleted the patch-1 branch June 10, 2024 08:11
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.

2 participants