Github user danny0405 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2619#discussion_r178454116
--- Diff: conf/defaults.yaml ---
@@ -195,7 +195,7 @@ worker.log.level.reset.poll.secs: 30
# control how many worker receiver threads we need per worker
topology.worker.receiver.thread.count: 1
-# Executor metrics reporting interval.
+# Executor metrics reporting interval, keep default val same with
topology.builtin.metrics.bucket.size.secs
--- End diff --
Because:
1. the internal metrics interval is set up by conf:
topology.builtin.metrics.bucket.size.secs, we should keep sync with it
2. Now use ZK to store our metrics and the ui only show built in metrics,
the metrics consumer's collecting interval are also default to 60 seconds
---
