[
https://issues.apache.org/jira/browse/STORM-766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14502872#comment-14502872
]
ASF GitHub Bot commented on STORM-766:
--------------------------------------
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/529#discussion_r28690473
--- Diff: storm-core/src/storm.thrift ---
@@ -153,6 +153,7 @@ struct SupervisorSummary {
3: required i32 num_workers;
4: required i32 num_used_workers;
5: required string supervisor_id;
+ 6: required string version;
--- End diff --
This needs to be optional, to maintain rolling upgrade compatibility.
> Supervisor summary should include the version.
> ----------------------------------------------
>
> Key: STORM-766
> URL: https://issues.apache.org/jira/browse/STORM-766
> Project: Apache Storm
> Issue Type: Bug
> Affects Versions: 0.10.0
> Reporter: Parth Brahmbhatt
> Assignee: Sanket Chintapalli
> Priority: Minor
> Fix For: 0.10.0
>
>
> With the support for rolling upgrade, different nodes in the cluster can run
> different versions of storm. We should include the version in
> SupervisorSummary just like NimbusSummary so admins can identify nodes that
> needs upgrading/downgrading from UI.
> As part of this change I will also add a supervisor/log link in the ui.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
