[GitHub] tysonnorris commented on issue #2795: enable concurrent activation processing

[gi...@git.apache.org]

tysonnorris commented on issue #2795: enable concurrent activation processing
URL: https://github.com/apache/incubator-openwhisk/pull/2795#issuecomment-331508825
 
 
   @markusthoemmes I think you mean to keep containers in freePool, and only move to busyPool when reaching max? (I'm not sure that WorkerState is used anymore?)
   
   If we do that, then ContainerPool.remove needs to pay attention to the concurrent invocations, and there will be potentially a lot of shuffling between freePool and busyPool when traffic hovers around max. In the other case (where container moves to busy after 1), there is potentially a lot of shuffling when there is a stream of sequential events (1 at a time), so maybe this doesn't matter. 
   
   I'll see if I can change the remove to respect the concurrent activation count and avoid complicating the scheduling as you describe.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services