Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2761#discussion_r202115684
--- Diff: storm-client/src/storm.thrift ---
@@ -206,10 +206,8 @@ struct NimbusSummary {
struct ClusterSummary {
1: required list<SupervisorSummary> supervisors;
- //@deprecated, please use nimbuses.uptime_secs instead.
- 2: optional i32 nimbus_uptime_secs = 0;
- 3: required list<TopologySummary> topologies;
- 4: required list<NimbusSummary> nimbuses;
+ 2: required list<TopologySummary> topologies;
+ 3: required list<NimbusSummary> nimbuses;
--- End diff --
https://diwakergupta.github.io/thrift-missing-guide/ describes some of this.
Specifically look at the bottom part about Versioning/Compatibility.
But there is some of it spread throughout the doc.
---
