Github user mjsax commented on a diff in the pull request:
https://github.com/apache/flink/pull/1398#discussion_r45726840
--- Diff:
flink-contrib/flink-storm/src/main/java/org/apache/flink/storm/wrappers/BoltWrapper.java
---
@@ -75,11 +78,13 @@
*
* @param bolt
* The Storm {@link IRichBolt bolt} to be used.
+ * @param componentId
+ * @param streamId
* @throws IllegalArgumentException
* If the number of declared output attributes is not with
range [0;25].
*/
- public BoltWrapper(final IRichBolt bolt) throws
IllegalArgumentException {
- this(bolt, null, (Collection<String>) null);
+ public BoltWrapper(final IRichBolt bolt, String componentId, String
streamId) throws IllegalArgumentException {
+ this(bolt, componentId, streamId, null, (Collection<String>)
null);
--- End diff --
This makes the interface quite hard to use and should not be necessary. The
`TopologyContext` created in `open()` contains this information.
---
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.
---
