Mailing-List: contact dev-help@brooklyn.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@brooklyn.apache.org
From: drigodwin <git@git.apache.org>
To: dev@brooklyn.apache.org
Reply-To: dev@brooklyn.apache.org
References: <git-pr-231-brooklyn-docs@git.apache.org>
In-Reply-To: <git-pr-231-brooklyn-docs@git.apache.org>
Subject: [GitHub] brooklyn-docs pull request #231: document the new ElectPrimaryPolicy and rel...
Content-Type: text/plain
Message-Id: <20171106155702.D218EDFE15@git1-us-west.apache.org>
Date: Mon,  6 Nov 2017 15:57:02 +0000 (UTC)
archived-at: Mon, 06 Nov 2017 15:57:06 -0000

Github user drigodwin commented on a diff in the pull request:

    https://github.com/apache/brooklyn-docs/pull/231#discussion_r149119018
  
    --- Diff: guide/blueprints/policies.md ---
    @@ -78,11 +78,18 @@ example of these working together.
     - org.apache.brooklyn.policy.ha.ServiceReplacer
     
     The ServiceReplacer attaches to a DynamicCluster and replaces a failed member in response to 
    -`ha.entityFailed` (or other configurable sensor).  
    +`ha.entityFailed` (or other configurable sensor) as typically emitted by the ServiceFailureDetector enricher.  
     The [introduction to policies]({{book.path.docs}}/start/policies.md) shows a worked 
     example of this policy in use.
     
     
    +#### ServiceFailureDetector Enricher
    +
    +- org.apache.brooklyn.policy.ha.ServiceFailureDetector
    +
    +TODO
    --- End diff --
    
    I don't think we should include TODOs in the docs


---