Github user m4rkmckenna commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/800#discussion_r134481212
--- Diff:
policy/src/main/java/org/apache/brooklyn/policy/autoscaling/AutoScalerPolicy.java
---
@@ -458,6 +459,25 @@ public void onEvent(SensorEvent<Number> event) {
}
};
+ /**
+ * This should usually be a no-op as the min and max pool sizes are
taken into account when
+ * an automatic resize event occurs, however this covers the special
case where the user
+ * has manually resized the pool
+ */
+ private SensorEventListener<? super Integer> poolEventHandler = new
SensorEventListener<Integer>() {
+ @Override
+ public void onEvent(SensorEvent<Integer> event) {
+ Sensor<Integer> sensor = event.getSensor();
+ assert sensor.equals(DynamicCluster.GROUP_SIZE);
--- End diff --
Is `assert` not removed at runtime??
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
