Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/889#issuecomment-169690819
Looks OK to me overall.
It would be nice to have a follow-on Jira Issue or Sub-Task to
remove/revert the changes we need to bootstrap the testing for this (dev-tools,
etc.).
> I have tested this by running a 0.10.0-SNAPSHOT storm starter topologies
against a cluster running this patch (along with the move_package.sh executed).
We found that with word_count, the .clj source file had been removed from
the transformed jar, but we expected that this .clj would be present, but
transformed.
---
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.
---