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

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

Github user unsleepy22 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1445#discussion_r70205708
  
    --- Diff: storm-core/src/jvm/org/apache/storm/daemon/StormCommon.java ---
    @@ -539,10 +542,34 @@ protected IAuthorizer 
mkAuthorizationHandlerImpl(String klassName, Map conf) thr
                     if (aznHandler != null) {
                         aznHandler.prepare(conf);
                     }
    -                LOG.debug("authorization class name:{}, class:{}, 
handler:{}",klassName, aznClass, aznHandler);
    +                LOG.debug("authorization class name:{}, class:{}, 
handler:{}", klassName, aznClass, aznHandler);
                 }
             }
     
             return aznHandler;
         }
    +
    +    public static WorkerTopologyContext makeWorkerContext(Map<String, 
Object> workerData) {
    +        try {
    +            StormTopology stormTopology = (StormTopology) 
workerData.get("system-topology");
    --- End diff --
    
    addressed, moved them into Constants class


> port backtype.storm.daemon.executor to java
> -------------------------------------------
>
>                 Key: STORM-1277
>                 URL: https://issues.apache.org/jira/browse/STORM-1277
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Cody
>              Labels: java-migration, jstorm-merger
>
> https://github.com/apache/storm/tree/jstorm-import/jstorm-core/src/main/java/com/alibaba/jstorm/task
>  kind of.  Tasks and executors are combined in jstorm.



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

Reply via email to