You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by "John Fang (JIRA)" <ji...@apache.org> on 2015/11/27 13:50:11 UTC

[jira] [Comment Edited] (STORM-1279) port backtype.storm.daemon.supervisor to java

    [ https://issues.apache.org/jira/browse/STORM-1279?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15029820#comment-15029820 ] 

John Fang edited comment on STORM-1279 at 11/27/15 12:50 PM:
-------------------------------------------------------------

I review the superviosor of Storm&JStorm. JStorm’s supervisor is almostly same with Storm’s supervisor. So we can look JStorm’s supervisor for reference when we are translating supervisor in phrase 1. But I personally hope Storm’s supervisor can combine  "synchronize-supervisor" and "sync-processes" in phrase 1. Please refer pdf  for the reasons. Of course, none of this is set in stone, just presented for discussion. Any and all comments are welcome.


was (Author: johnbaba):
I review the superviosor of Storm&JStorm. JStorm’s supervisor is almostly same with Storm’s supervisor. So we can look JStorm’s supervisor for reference when we are translating supervisor in phrase 1. But I personally hope Storm’s supervisor can combine  "synchronize-supervisor" and "sync-processes" in phrase 1. Please refer pdf  for the reasons. Of course, it's just my mind. If you have anything in your mind, I want you to come out with it.

> port backtype.storm.daemon.supervisor to java
> ---------------------------------------------
>
>                 Key: STORM-1279
>                 URL: https://issues.apache.org/jira/browse/STORM-1279
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: John Fang
>              Labels: java-migration, jstorm-merger
>         Attachments: Discussion about supervisor.pdf
>
>
> https://github.com/apache/storm/tree/jstorm-import/jstorm-core/src/main/java/com/alibaba/jstorm/daemon/supervisor as an example
> backtype.storm.event usage should be replaced with built-in java threadpools.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)