[
https://issues.apache.org/jira/browse/STORM-1129?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15217525#comment-15217525
]
ASF GitHub Bot commented on STORM-1129:
---------------------------------------
Github user abhishekagarwal87 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1277#discussion_r57843883
--- Diff: storm-core/src/ui/public/component.html ---
@@ -176,7 +176,7 @@
var loc = $(row[3])[0]; // logviewer URL
return '<input type="checkbox" class="workerActionCheckbox"'+
'id="'+checkboxId+'"
value="'+host_port+'"'+checkedString+'/> '+
- '<a
href="'+loc.protocol+'//'+loc.host+'/dumps/'+topologyId+'/'+
+ '<a
href="'+loc.protocol+'//'+loc.host+'/dumps/'+topologyName+'/'+
--- End diff --
should be left as it is.
> Storm should use topology name instead of ids for url in storm UI.
> ------------------------------------------------------------------
>
> Key: STORM-1129
> URL: https://issues.apache.org/jira/browse/STORM-1129
> Project: Apache Storm
> Issue Type: Bug
> Components: storm-core
> Reporter: Priyank Shah
> Assignee: Priyank Shah
>
> Currently, in storm UI details about a topology can be viewed at a URL which
> has a topology id as a query parameter. When a topology is updated and
> redeployed a new id is assigned by storm and existing URL(and any bookmarks
> relying on it) for the topology do not work since the id has changed. We
> should change it so that topology name is used instead of id.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
