Github user revans2 commented on a diff in the pull request:
--- Diff: storm-server/src/main/java/org/apache/storm/DaemonConfig.java ---
@@ -1040,6 +1039,13 @@
// and log an error message, so just validating this as a String for
public static final String STORM_METRIC_STORE_CLASS =
+ * Class implementing WorkerMetricsProcessor.
--- End diff --
It might be good to mention here in the javadocs that this is intended to
be run on the supervisor.