Github user aledsage commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/378#issuecomment-66265986
I think "None" is fine for now. It is the responsibility of the effector's
description (and parameter's description) to say what "None" means. For
example, its meaning when deploying for the first time is different from when
calling start() to the start the process again.
As an aside, if restarting a stopped process then I now think we probably
want to call `restart()` rather than `start()`. This is because `start()` will
go through the install+customize+launch steps, where customize could overwrite
config files with their defaults and could also run init scripts (e.g. to
initialise the database, which could cause data loss!).
Merging.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
