[GitHub] rabbah commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
rabbah commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412188985 See https://github.com/apache/incubator-openwhisk/blob/f6046721acd801ea1777cb5cb040dfaa6922a18e/tests/src/test/s

[GitHub] rabbah commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
rabbah commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412188134 I think there is such a test I’ll try to find it. For the second part I don’t think it matters. ---

[GitHub] rabbah commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
rabbah commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412024187 It’s better - would you want to reuse the container if it timed out? You’d end up with a concurrent activation for e