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

    https://github.com/apache/flink/pull/1398#discussion_r45971476
  
    --- Diff: 
flink-contrib/flink-storm/src/main/java/org/apache/flink/storm/wrappers/StormTuple.java
 ---
    @@ -44,16 +45,30 @@
        /** The schema (ie, ordered field names) of the tuple */
        private final Fields schema;
     
    +   private final int taskId;
    +   private final String producerStreamId;
    +   private final MessageId id;
    +   private final String producerComponentId;
    +
    +
    +   /**
    +    * Constructor which sets defaults for producerComponentId, taskId, and 
componentID
    +    * @param flinkTuple the Flink tuple
    +    * @param schema The schema of the storm fields
    +    */
    +   StormTuple(final IN flinkTuple, final Fields schema) {
    +           this(flinkTuple, schema, -1, "testStream", "componentID");
    +   }
    --- End diff --
    
    Fair enough, I use default id constants now.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to