[ 
https://issues.apache.org/jira/browse/STORM-441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14306072#comment-14306072
 ] 

ASF GitHub Bot commented on STORM-441:
--------------------------------------

Github user knusbaum commented on the pull request:

    https://github.com/apache/storm/pull/237#issuecomment-72950372
  
    Bootstrap is no more!



> 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)

Reply via email to