You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@storm.apache.org by "Erik Weathers (JIRA)" <ji...@apache.org> on 2017/12/19 04:08:00 UTC

[jira] [Updated] (STORM-2690) resurrect invocation of ISupervisor.assigned() & make Supervisor.launchDaemon() accessible

     [ https://issues.apache.org/jira/browse/STORM-2690?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Erik Weathers updated STORM-2690:
---------------------------------
    Summary: resurrect invocation of ISupervisor.assigned() & make Supervisor.launchDaemon() accessible  (was: resurrect invocation of ISupervisor.assigned() & make )

> resurrect invocation of ISupervisor.assigned() & make Supervisor.launchDaemon() accessible
> ------------------------------------------------------------------------------------------
>
>                 Key: STORM-2690
>                 URL: https://issues.apache.org/jira/browse/STORM-2690
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-core
>    Affects Versions: 2.0.0, 1.1.0, 1.0.3, 1.0.4, 1.1.1
>            Reporter: Erik Weathers
>            Assignee: Erik Weathers
>            Priority: Minor
>
> As [discussed in STORM-2018|https://issues.apache.org/jira/browse/STORM-2018?focusedCommentId=16108307&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16108307], that change subtly broke the storm-mesos integration framework because of the removal of the invocation of [{{ISupervisor.assigned()}}|https://github.com/apache/storm/blob/v1.0.4/storm-core/src/jvm/org/apache/storm/scheduler/ISupervisor.java#L44].
> So this ticket is tracking the reinstatement of that invocation from the supervisor core code.
> Also, the [{{launchDaemon()}}|https://github.com/apache/storm/blob/v1.0.4/storm-core/src/jvm/org/apache/storm/daemon/supervisor/Supervisor.java#L248] method of the {{Supervisor}} is not public, so we had to use reflection to allow calling it from the storm-mesos integration.  That should be changed too.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)