Github user mjsax commented on a diff in the pull request:
https://github.com/apache/flink/pull/1398#discussion_r45970656
--- 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 --
Ok. I guess it would make sense to use Storm's `Utils.DEFAULT_STREAM_ID`
here? And maybe add a `public final static String DEFAULT_OPERATOR_ID` variable
to `StormTuple`? What about using "defaultID" or "unspecified" instead of
"componentID" or similar? Just to make it clear if the name shows up in the UI?
---
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.
---
