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

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

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

    https://github.com/apache/storm/pull/1249#discussion_r57380470
  
    --- Diff: storm-core/src/jvm/org/apache/storm/daemon/Task.java ---
    @@ -0,0 +1,247 @@
    +/**
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + * http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.storm.daemon;
    +
    +import org.apache.storm.Config;
    +import org.apache.storm.Thrift;
    +import org.apache.storm.generated.Bolt;
    +import org.apache.storm.generated.ComponentObject;
    +import org.apache.storm.generated.JavaObject;
    +import org.apache.storm.generated.ShellComponent;
    +import org.apache.storm.generated.SpoutSpec;
    +import org.apache.storm.generated.StateSpoutSpec;
    +import org.apache.storm.generated.StormTopology;
    +import org.apache.storm.grouping.LoadMapping;
    +import org.apache.storm.hooks.ITaskHook;
    +import org.apache.storm.hooks.info.EmitInfo;
    +import org.apache.storm.metric.api.IMetric;
    +import org.apache.storm.spout.ShellSpout;
    +import org.apache.storm.stats.CommonStats;
    +import org.apache.storm.task.ShellBolt;
    +import org.apache.storm.task.TopologyContext;
    +import org.apache.storm.task.WorkerTopologyContext;
    +import org.apache.storm.tuple.Fields;
    +import org.apache.storm.tuple.Tuple;
    +import org.apache.storm.tuple.TupleImpl;
    +import org.apache.storm.utils.ConfigUtils;
    +import org.apache.storm.utils.Utils;
    +import org.slf4j.Logger;
    +import org.slf4j.LoggerFactory;
    +
    +import java.io.IOException;
    +import java.util.ArrayList;
    +import java.util.Collections;
    +import java.util.List;
    +import java.util.Map;
    +import java.util.concurrent.Callable;
    +
    +public class Task {
    +
    +    private static final Logger LOG = LoggerFactory.getLogger(Task.class);
    +
    +    private Map executorData;
    +    private Map workerData;
    +    private TopologyContext systemTopologyContext;
    +    private TopologyContext userTopologyContext;
    +    private WorkerTopologyContext workerTopologyContext;
    +    private LoadMapping loadMapping;
    +    private Integer taskId;
    +    private String componentId;
    +    private Object taskObject;
    +    private Map stormConf;
    +    private Callable<Boolean> emitSampler;
    +    private CommonStats executorStats;
    +    private Map<String, Map<String, Grouper>> streamComponentToGrouper;
    +    private Object builtInMetrics;      //TODO
    --- End diff --
    
    I don't think we want any TODO comments in the code. I'm guessing 
builtin-metrics should be merged before this?


> port backtype.storm.daemon.task to java
> ---------------------------------------
>
>                 Key: STORM-1271
>                 URL: https://issues.apache.org/jira/browse/STORM-1271
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Abhishek Agarwal
>              Labels: java-migration, jstorm-merger
>
> helper functions for task data and sending tuples



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

Reply via email to