Github user danny0405 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2433#discussion_r152717373
--- Diff: storm-client/src/jvm/org/apache/storm/stats/StatsUtil.java ---
@@ -1589,18 +1570,16 @@ public static ComponentPageInfo aggCompExecsStats(
if (lastReportedTime != null) {
reportedTime = lastReportedTime;
} else {
- reportedTime = 0;
+ reportedTime = lastReportedTime = 0;
}
}
if (lastNimbusTime == null ||
!reportedTime.equals(lastReportedTime)) {
lastNimbusTime = Time.currentTimeSecs();
}
- ret.put("is-timed-out", Time.deltaSecs(lastNimbusTime) >= timeout);
--- End diff --
yes, it's safe we will refresh the is-timed-out when we make assignments
---
