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

    https://github.com/apache/storm/pull/2319#discussion_r138657756
  
    --- Diff: storm-core/src/clj/org/apache/storm/cluster.clj ---
    @@ -123,6 +133,7 @@
     (def SUPERVISORS-SUBTREE (str "/" SUPERVISORS-ROOT))
     (def WORKERBEATS-SUBTREE (str "/" WORKERBEATS-ROOT))
     (def BACKPRESSURE-SUBTREE (str "/" BACKPRESSURE-ROOT))
    +(def LEADERINFO-SUBTREE (str "/" LEADERINFO-ROOT))
    --- End diff --
    
    Is this needed?  we already have a way of getting the leader that is built 
into the client.  Why do we need a second way?


---

Reply via email to