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

Reduce default pool size to something else #110

Open
krowvin opened this issue Oct 24, 2024 · 2 comments
Open

Reduce default pool size to something else #110

krowvin opened this issue Oct 24, 2024 · 2 comments

Comments

@krowvin
Copy link

krowvin commented Oct 24, 2024

adapter = adapters.HTTPAdapter(pool_connections=100, pool_maxsize=100)

Suggest 15 based on tests done

User experienced this error
image

Edit: this is @mdigiorno1 output, I couldn't tag you earlier

@krowvin
Copy link
Author

krowvin commented Oct 24, 2024

@DanielTOsborne had some concerns on too many threads here
USACE-WaterManagement/repgen5#35 (comment)

@mdigiorno1
Copy link

This is the code I ran that gave the error:
image
The first call had 168 tsids and causes the warnings. There are no warnings from the second call (67 tsids).

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

No branches or pull requests

2 participants