Github user aledsage commented on the issue:

    https://github.com/apache/brooklyn-server/pull/155
  
    On balance, I agree with @ahgittin that we should revert this, and first 
merge https://github.com/apache/brooklyn-server/pull/480
    
    The changes in this PR were suggested on the mailing list as part of 
@grkvlt's message "[PROPOSAL] Enabling Effective Effectors" on 30/05/2016, but 
it wasn't properly discussed there.
    
    After reverting this, we need to kick off that discussion on the mailing 
list again. We need to more clearly describe the use-case that motivated this 
change. We can then discuss the philosophy behind it.
    
    For clarification, I believe the intent for the CA-server use-case was that 
the effector call happens *only once*; subsequent queries for the config key 
gets the value from that initial invocation (or blocks until the effector 
returns, if another thread is still executing it). But let's discuss that more 
on the mailing list.


---
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.
---

Reply via email to