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.
---