[
https://issues.apache.org/jira/browse/STORM-1614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15191457#comment-15191457
]
ASF GitHub Bot commented on STORM-1614:
---------------------------------------
Github user zhuoliu commented on a diff in the pull request:
https://github.com/apache/storm/pull/1206#discussion_r55881939
--- Diff: storm-core/src/clj/org/apache/storm/cluster.clj ---
@@ -73,6 +73,7 @@
(teardown-topology-errors! [this storm-id])
(heartbeat-storms [this])
(error-topologies [this])
+ (backpressure-topologies [this])
--- End diff --
The name - "backpressure-topologies" is a little confusing (people may
think "backpressure" as verb) since it just returns the current topologies' zk
nodes (ids) under backpressure root. We might rename it as something like
"get-backpressure-topologies-ids".
> Clean backpressure zk node in do-cleanup
> ----------------------------------------
>
> Key: STORM-1614
> URL: https://issues.apache.org/jira/browse/STORM-1614
> Project: Apache Storm
> Issue Type: Bug
> Components: storm-core
> Reporter: Alessandro Bellina
> Assignee: Alessandro Bellina
> Priority: Minor
>
> Currently the backpressure node is being removed in killTopologyWithOpts in
> nimbus. Remove instead like the other ZK nodes in do-cleanup for inactive
> topos.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
