[
https://issues.apache.org/jira/browse/STORM-436?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Joseph Evans updated STORM-436:
--------------------------------------
Labels: newbie (was: )
> Clean up Clojure namespace declarations
> ---------------------------------------
>
> Key: STORM-436
> URL: https://issues.apache.org/jira/browse/STORM-436
> Project: Apache Storm
> Issue Type: Improvement
> Affects Versions: 0.9.2-incubating
> Reporter: Daniel Compton
> Priority: Minor
> Labels: newbie
>
> Some of the Clojure namespace declarations in the storm project are messy and
> non-idiomatic.
> https://github.com/apache/incubator-storm/blob/master/storm-core/src/clj/backtype/storm/ui/core.clj#L18-L38
> is a good example of this.
> There are a few things I'd like to improve:
> 1. Coalesce multiple use/require/import's into a single use/require/import.
> 2. Order imports use, require, import.
> 3. Optionally, replacing use with some mix of
> * [... :refer :all]
> * Referring just the vars that are used
> * Qualifying the namespace imports
> Is there a reason why the namespaces were done in the way they have been?
> What would be the preferred way to do 3?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
