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

ASF GitHub Bot commented on STORM-1279:
---------------------------------------

Github user hustfxj commented on the pull request:

    https://github.com/apache/storm/pull/1184#issuecomment-193738634
  
    @zhuoliu @abhishekagarwal87  Thank you. I will consider your suggestions, 
and adjust class hierarchy. Of course, I hope others can help review the 
supervisor and give me some comments. Then I will revise the codes together in 
order to avoid to different suggestions and repeat modifications. Thank you 
again.


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

Reply via email to