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