Github user srdo commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2761#discussion_r202126575
  
    --- 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 --
    
    Thanks. Is it worth it to link to in DEVELOPER,md, or somewhere else do you 
think?


---

Reply via email to