Github user neykov commented on a diff in the pull request:
https://github.com/apache/incubator-brooklyn/pull/705#discussion_r33141815
--- Diff:
policy/src/main/java/brooklyn/enricher/TimeWeightedDeltaEnricher.java ---
@@ -41,6 +42,8 @@
* presented as a units/timeUnit based on the event timing.
* <p>
* NB for time (e.g. "total milliseconds consumed") use {@link
TimeFractionDeltaEnricher}
+ * <p>
+ * See also {@link YamlTimeWeightedDeltaEnricher} designed for use from
YAML.
--- End diff --
If both this and the YAML alternative are going to be used long-term, why
not re-use the code? Alternatively deprecate this one?
Analogous comment for `RollingTimeWindowMeanEnricher` would be nice.
---
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.
---
