markusthoemmes commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r184601638
########## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ########## @@ -107,6 +106,7 @@ class ContainerProxy( implicit val ec = context.system.dispatcher implicit val logging = new AkkaLogging(context.system.log) var rescheduleJob = false // true iff actor receives a job but cannot process it because actor will destroy itself + var activeActivationCount = 0 Review comment: WDYT about making this part of `WarmedData` or even `ContainerData`? That would eschew the need for the `ActivationCounter` and make other parts easier as well (I'll ad comments for posterity) ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services