Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2433#discussion_r173213537 --- Diff: storm-client/src/jvm/org/apache/storm/cluster/IStormClusterState.java --- @@ -233,23 +249,11 @@ * @return the id of the topology or null if it is not alive. */ default Optional<String> getTopoId(final String topologyName) { --- End diff -- I am a little confused why we need 2 methods that appear to do the exact same thing. Could we try to have just one of them?
---