[
https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14146766#comment-14146766
]
ASF GitHub Bot commented on STORM-441:
--------------------------------------
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/237#issuecomment-56724905
Looks good, now we need to do the same for all of the tests that also use
bootstrap
> Remove bootstrap macro from Clojure codebase
> --------------------------------------------
>
> Key: STORM-441
> URL: https://issues.apache.org/jira/browse/STORM-441
> Project: Apache Storm
> Issue Type: Improvement
> Reporter: Dane Hammer
> Priority: Trivial
>
> The bootstrap macro in backtype.storm.bootstrap is purely a convenience for
> importing/using/requiring a large number of dependencies, but it's not used
> for anything else. It removes those imports/uses/requires from the namespace
> form, making it harder to track down where a definition is coming from, which
> defeats some IDE tools.
> I propose removing it entirely, making the Clojure part of the codebase more
> readable and updated to current conventions.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
