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

    https://github.com/apache/brooklyn-library/pull/126#discussion_r142680516
  
    --- Diff: 
software/nosql/src/main/java/org/apache/brooklyn/entity/nosql/mongodb/sharding/MongoDBRouterClusterImpl.java
 ---
    @@ -56,12 +60,17 @@ public void start(Collection<? extends Location> 
locations) {
         
         public static class MemberTrackingPolicy extends 
AbstractMembershipTrackingPolicy {
             @Override protected void onEntityEvent(EventType type, Entity 
member) {
    +            defaultHighlightAction(type, entity);
    --- End diff --
    
    Not needed as this policy extends `AbstractMembershipTrackingPolicy ` and 
`onEntityEvent` [already call 
`defaultHighlightAction`](https://github.com/apache/brooklyn-server/blob/master/core/src/main/java/org/apache/brooklyn/entity/group/AbstractMembershipTrackingPolicy.java#L242)


---

Reply via email to