[ 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)