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