brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aledsage <...@git.apache.org>
Subject [GitHub] brooklyn-server issue #155: [WIP] Add DSL support for calling effectors in Y...
Date Wed, 01 Mar 2017 11:06:05 GMT
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message