[GitHub] dubee commented on issue #2855: Cache database attachments

2018-05-09 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-387806564 @csantanapr, just need a 👍. This is an automated message from the Apache Git Servi

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-05-08 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-387483534 @rabbah, looks like caching is enabled/disabled on the `WhiskAction` singleton level, which is common code shared among the controller and invoker.

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-20 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-383188018 PG4 1600 🔵 This is an automated message from the Apache Git Service. To respond t

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-19 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-382812283 PG4 1600 ⏳ This is an automated message from the Apache Git Service. To respond t

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-18 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-382407416 @rabbah, @markusthoemmes, thoughts on this one? Seems like there's a fairly nice performance boost for invoking Java actions.

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-16 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381743878 I'm not very concerned about the controller caching Java actions. Mostly the action invoke path is used in the controller. With that action invoke

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-16 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381743878 I'm not very concerned about the controller caching Java actions. Mostly the action invoke path is used in the controller. With that action invoke

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-16 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381743878 I'm not very concerned about the controller caching Java actions. Mostly the action invoke path is used in the controller. With that action invoke

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-16 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381743878 I'm not very concerned about the controller caching Java actions. Mostly the action invoke path is used in the controller. With that action invoke

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-16 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381727877 @bbrowning, how many invokers are you using in your test cluster? This is an autom

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-15 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381200828 @bbrowning, do you think `key` needs to be change in `attach` and `put` as well?

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-13 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381200828 @bbrowning, do you think `key` needs to be change in `attach` and `put` as well?

[GitHub] dubee commented on issue #2855: Cache database attachments

2018-04-13 Thread GitBox
dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-381187596 @bbrowning, thanks for the feedback! This is an automated message from the Apache