You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2015/02/04 23:08:34 UTC

[jira] [Commented] (STORM-441) Remove bootstrap macro from Clojure codebase

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