Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2619#discussion_r178453508 --- 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 -- Nit: Could you add to the comment the reason why these must be kept in sync?
---