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

    https://github.com/apache/storm/pull/2759#discussion_r201999379
  
    --- Diff: storm-client/src/jvm/org/apache/storm/daemon/StormCommon.java ---
    @@ -85,11 +84,6 @@ public static StormCommon setInstance(StormCommon 
common) {
             return oldInstance;
         }
     
    -    @Deprecated
    --- End diff --
    
    The method isn't really very useful. It's just as easy to just call 
getTopoId on the StormClusterState yourself, rather than using this method. I 
think that's a good enough reason to remove it.


---

Reply via email to