brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From grkvlt <...@git.apache.org>
Subject [GitHub] brooklyn-server pull request #833: Fix PeriodicEffectorPolicy
Date Wed, 20 Sep 2017 10:51:44 GMT
Github user grkvlt commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/833#discussion_r139935683
  
    --- Diff: policy/src/main/java/org/apache/brooklyn/policy/action/AbstractScheduledEffectorPolicy.java
---
    @@ -101,15 +102,15 @@
     
         public static final ConfigKey<Boolean> RUNNING = ConfigKeys.builder(Boolean.class)
                 .name("running")
    -            .description("Set if the executor has started")
    +            .description("[INTERNAL] Set if the executor has started")
                 .defaultValue(Boolean.FALSE)
                 .reconfigurable(true)
                 .build();
     
         public static final ConfigKey<List<Long>> SCHEDULED = ConfigKeys.builder(new
TypeToken<List<Long>>() { })
                 .name("scheduled")
                 .description("List of all scheduled execution start times")
    -            .defaultValue(Lists.newCopyOnWriteArrayList())
    +            .defaultValue(ImmutableList.of())
    --- End diff --
    
    We can use the non-COW version because you just re-set the config with a mutated copy,
right?


---

Mime
View raw message