[
https://issues.apache.org/jira/browse/STORM-2690?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Erik Weathers updated STORM-2690:
---------------------------------
Description:
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.
was:
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.
> resurrect invocation of ISupervisor.assigned() & make
> ------------------------------------------------------
>
> 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)
