Reduce default pool size to something else

[krowvin]

cwms-python/cwms/api.py

Line 46 in f1f1a59

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

Suggest 15 based on tests done

• Feature/threading USACE-WaterManagement/repgen5#35 (comment)

User experienced this error

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

[krowvin]

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

[mdigiorno1]

This is the code I ran that gave the error:

The first call had 168 tsids and causes the warnings. There are no warnings from the second call (67 tsids).

[krowvin]

@mdigiorno1

I do not recall - but was it dropping the remaining threads over 100 for your code?

If these were just warnings I suspect it is because the pool size is hard set at 100 in the source code

These days I think cwms-python supports threading/async calls with

• cwms-python/cwms/timeseries/timeseries.py

  Line 12 in 2684dcf

  def get_multi_timeseries_df(

If you wanted to try that instead of asyncio