You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@storm.apache.org by bo...@apache.org on 2018/09/25 17:22:04 UTC

[3/3] storm git commit: STORM-3234: Addressed review comments

STORM-3234: Addressed review comments


Project: http://git-wip-us.apache.org/repos/asf/storm/repo
Commit: http://git-wip-us.apache.org/repos/asf/storm/commit/9d083fa9
Tree: http://git-wip-us.apache.org/repos/asf/storm/tree/9d083fa9
Diff: http://git-wip-us.apache.org/repos/asf/storm/diff/9d083fa9

Branch: refs/heads/master
Commit: 9d083fa919e0000d9b74938b80459c8833b116b3
Parents: 5334e0c
Author: Robert Evans <ev...@yahoo-inc.com>
Authored: Tue Sep 25 11:53:50 2018 -0500
Committer: Robert Evans <ev...@yahoo-inc.com>
Committed: Tue Sep 25 11:53:50 2018 -0500

----------------------------------------------------------------------
 docs/ClusterMetrics.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/storm/blob/9d083fa9/docs/ClusterMetrics.md
----------------------------------------------------------------------
diff --git a/docs/ClusterMetrics.md b/docs/ClusterMetrics.md
index d32d0b1..17c4ca1 100644
--- a/docs/ClusterMetrics.md
+++ b/docs/ClusterMetrics.md
@@ -20,7 +20,7 @@ These are metrics that come from the active nimbus instance and report the state
 
 | Metric Name | Type | Description |
 |-------------|------|-------------|
-| cluster:num-nimbus-leaders | gauge | Number of nimbuses marked as a leader. This should really only ever be 1 in a health cluster, or 0 for a short period of time while a failover happens. |
+| cluster:num-nimbus-leaders | gauge | Number of nimbuses marked as a leader. This should really only ever be 1 in a healthy cluster, or 0 for a short period of time while a failover happens. |
 | cluster:num-nimbuses | gauge | Number of nimbuses, leader or standby. |
 | cluster:num-supervisors | gauge | Number of supervisors. |
 | cluster:num-topologies | gauge | Number of topologies. |
@@ -87,7 +87,7 @@ These are metrics that are specific to a nimbus instance.  In many instances onl
 | nimbus:num-killTopologyWithOpts-calls | meter | calls to killTopologyWithOpts thrift method includes calls to killTopology. |
 | nimbus:num-launched | meter | number of times a nimbus was launched |
 | nimbus:num-lost-leadership | meter | number of times this nimbus lost leadership |
-| nimbus:num-negative-resource-events | meter | any time a resource goes negative (either CPU or Memory)  Not consistent as it is used for internal calculations that may go negative and does not represent over scheduling of resources. |
+| nimbus:num-negative-resource-events | meter | Any time a resource goes negative (either CPU or Memory).  This metric is not ideal as it is measured in a data structure that is used for internal calculations that may go negative and not actually represent over scheduling of a resource. |
 | nimbus:num-net-executors-increase-per-scheduling | histogram | added executors minus removed executors after a scheduling run |
 | nimbus:num-net-slots-increase-per-scheduling | histogram | added slots minus removed slots after a scheduling run |
 | nimbus:num-rebalance-calls | meter | calls to rebalance thrift method. |