GitHub user priyank5485 opened a pull request:
https://github.com/apache/storm/pull/854
STORM-1129: Use topology name instead of id in UI calls.
Note that all-topologies-summary has been used to get topology id from a
topology name. That involves a few calls to zookeeper which is not ideal.
However, UI does not seem to take any significant performance hit. If needed we
can handle it possibly using one of the options below. Since its a separate
performance issue we can handle it in a separate JIRA.
1. Have nimbus thrift server cache summary for topologies so it does not
hit zookeeper every time we try to get topology id from name.
2. Update nimbus thrift api with a method that takes options and use that
to do only the minimal necessary interaction with zookeeper for a given option.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/priyank5485/storm STORM-1129
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/854.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 #854
----
commit 77b73af52bdc2d691ae172a1a2905a94395c3c70
Author: Priyank <[email protected]>
Date: 2015-11-04T00:20:50Z
STORM-1129: Use topology name instead of id in UI calls.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
