Github user dan-blanchard commented on the pull request:
https://github.com/apache/storm/pull/525#issuecomment-94457226
> It would be more safer to have a test, but since real TopologyContext is
made from deeply inside of executor creation, I'm not sure we can instantiate
several TopologyContexts manually from test.
Yeah, I thought the same thing, but like you said, I wasn't sure how to
manually setup TopologyContexts.
I think the only thing that needs to be tested at this point is that it
works for fields groupings, which it should.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
