[
https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14104725#comment-14104725
]
ASF GitHub Bot commented on STORM-441:
--------------------------------------
GitHub user knusbaum opened a pull request:
https://github.com/apache/incubator-storm/pull/237
STORM-441: Remove bootstrap macro from Clojure codebase
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/knusbaum/incubator-storm STORM-441
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/237.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 #237
----
commit 6c504613c5cb7dead19c59f17f87d7d8179a0671
Author: Kyle Nusbaum <[email protected]>
Date: 2014-08-20T22:29:51Z
Rid storm of bootstrap
----
> Remove bootstrap macro from Clojure codebase
> --------------------------------------------
>
> Key: STORM-441
> URL: https://issues.apache.org/jira/browse/STORM-441
> Project: Apache Storm (Incubating)
> 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.2#6252)
