davidsantiago/clojure-hbase — Issue #6: Callers of set-config should not have to wrap keys in a map literal
Author: rplevy
Created: 2012-03-05 13:42:21+00:00
State: open

Thanks for merging in my recent pull request.

Can I make one suggestion regarding the change you made, which you can feel free to ignore? If you're going to accept a map argument, consider instead the destructuring form {:keys [] :as settings}, which puts the whole map into settings (no keys specified in key vector).

See Library Coding Standards http://dev.clojure.org/display/design/Library+Coding+Standards "Callers should not have to wrap optional named arguments in a map literal"