[
https://issues.apache.org/jira/browse/STORM-599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14256209#comment-14256209
]
ASF GitHub Bot commented on STORM-599:
--------------------------------------
GitHub user d2r opened a pull request:
https://github.com/apache/storm/pull/356
[STORM-599] Use use nimbus's cached heartbeats rather than fetching again
from ZK
In my manual testing, this seems to cut the time to load a topology page by
around 50%.
This also removes some apparently unnecessary code.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/d2r/storm apache-use-hb-cache-for-topo-info
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/356.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #356
----
commit 3907b2d780b796e754bf3566d29567cff6e77679
Author: Derek Dagit <[email protected]>
Date: 2014-12-13T15:36:31Z
use the cache instead of downloading from zk
commit 40be5ddb94b674f8fcf2c804c7af96fbfe77776a
Author: Derek Dagit <[email protected]>
Date: 2014-12-13T15:36:51Z
Remove unnecessary code
----
> UI is slow due to fetching heartbeats from ZK
> ---------------------------------------------
>
> Key: STORM-599
> URL: https://issues.apache.org/jira/browse/STORM-599
> Project: Apache Storm
> Issue Type: Improvement
> Affects Versions: 0.9.3
> Reporter: Derek Dagit
> Assignee: Derek Dagit
> Priority: Minor
>
> The method getTopologyInfo fetches every heartbeat from ZooKeeper in order to
> gather statistics. The UI calls this method via thrift to render the
> topology page.
> When topologies are launched with thousands of executors, this fetching from
> ZK dramatically slows things down.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
