markusthoemmes commented on a change in pull request #3104: Add TID stride
configuration
URL:
https://github.com/apache/incubator-openwhisk/pull/3104#discussion_r160877665
##########
File path: common/scala/src/main/resources/application.conf
##########
@@ -82,4 +82,7 @@ whisk {
activations-ddoc = "whisks.v2"
activations-filter-ddoc = "whisks-filters.v2"
}
+
+ controller {
Review comment:
It feels a bit odd to add "controller" to the common configuration, does it?
Also you don't want to use camelCased but rather kebab-cased fieldnames,
otherwise the override through the env-convert script won't work.
Should we rather put this under a "transactions" key?
`whisk.transactions.strides`
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
