GitHub user revans2 opened a pull request:
https://github.com/apache/storm/pull/2072
STORM-2477: Add generics to Config types
Because this touches so much of the code to avoid having to constantly
upmerge I have a script that will transform most of the code automatically.
transform.sh . I don't plan on checking it in. If everyone is OK with the
change I will run the script and just check those changes in. The other
changes and small updates to make the result of running the script not produce
compile errors. Overall this drops more than 800 warnings from the code base
(according to eclipse).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/revans2/incubator-storm STORM-2477
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2072.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2072
----
commit dc6d2bbd540304552e9c56eba5d24eb0e4d0f74b
Author: Robert (Bobby) Evans <[email protected]>
Date: 2017-04-15T21:21:20Z
Fixed issues with generics to make script work
commit 0b088ce1716ffabb6bace2c58e39220f187c1769
Author: Robert (Bobby) Evans <[email protected]>
Date: 2017-04-16T00:17:27Z
Script to change add proper types to configs and make stormConf topoConf
----
---
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.
---
