reduce max threads to 30#336
Conversation
|
I would be interested to wait until the new ESGF-NG configurations from the JASMIN team are in place as it may be the case we can keep a high MAX_THREAD count. This could be important for high concurrency down the line for multiple users requesting reductions. I can run a simple MAX_THREAD v Run Time test to understand for https server so we can see what is optimal, rather than just patching with 30 at the moment. Thoughts/ opinions welcome |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #336 +/- ##
=======================================
Coverage 93.13% 93.13%
=======================================
Files 7 7
Lines 685 685
=======================================
Hits 638 638
Misses 47 47 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
indeed what I thought too! We can do tests via here 🍺 |
Description
Investigations in #335 show that we don't really take a performance hit from lowering the number of max threads used by Active, but we manage to coax the CEDA NGINX server to complete its work without issues (most of the times).
EDIT: it appears that test outcmes are pretty bad today - and, as @varsiha-sothilingam mentioned, we are going to wait to see the new CEDA NGINX storage performance via here.
Contributes towards avoiding #329
Closes #333 and #335
Before you get started
Checklist