Mailing-List: contact dev-help@brooklyn.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@brooklyn.apache.org
From: aledsage <git@git.apache.org>
To: dev@brooklyn.apache.org
Reply-To: dev@brooklyn.apache.org
References: <git-pr-155-brooklyn-server@git.apache.org>
In-Reply-To: <git-pr-155-brooklyn-server@git.apache.org>
Subject: [GitHub] brooklyn-server issue #155: [WIP] Add DSL support for calling effectors in Y...
Content-Type: text/plain
Message-Id: <20170301110605.4732BDFDE6@git1-us-west.apache.org>
Date: Wed,  1 Mar 2017 11:06:05 +0000 (UTC)
archived-at: Wed, 01 Mar 2017 11:06:07 -0000

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