Daniel Compton created STORM-436:
------------------------------------
Summary: Clean up Clojure namespace declarations
Key: STORM-436
URL: https://issues.apache.org/jira/browse/STORM-436
Project: Apache Storm (Incubating)
Issue Type: Improvement
Affects Versions: 0.9.2-incubating
Reporter: Daniel Compton
Priority: Minor
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.2#6252)
