markusthoemmes opened a new pull request #4843: Replace remaining usages of 
parallel collections with explicit concurrency.
URL: https://github.com/apache/openwhisk/pull/4843
 
 
   <!--- Provide a concise summary of your changes in the Title -->
   
   ## Description
   I broke this one out of the last PR because the concurrency looked a bit 
more tricky to get right on the surface. If I read the code correctly though, 
this should equally replicate the state we had before.
   
   Each individual task used way too big thread pools and the maximum amount of 
total work we do is `internalIters * 30` (cause the inner loops do 30 
iterations), thus a single fixed thread pool of that size should suffice and 
can be reused in all tasks.
   
   ## Related issue and scope
   Ref #4741 
   
   ## Checklist:
   <!--- Please review the points below which help you make sure you've covered 
all aspects of the change you're making. -->
   
   - [X] I signed an [Apache 
CLA](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md).
   - [X] I reviewed the [style 
guides](https://github.com/apache/openwhisk/wiki/Contributing:-Git-guidelines#code-readiness)
 and followed the recommendations (Travis CI will check :).
   - [X] I added tests to cover my changes.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to