Skip to content

Commit

Permalink
added the ability to supply comparator aliases when creating column f…
Browse files Browse the repository at this point in the history
…amilies
  • Loading branch information
Matt Stump committed Dec 22, 2011
1 parent d9a1ae5 commit 3130161
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
9 changes: 7 additions & 2 deletions src/clj_hector/ddl.clj
Original file line number Diff line number Diff line change
Expand Up @@ -57,14 +57,19 @@

(defn- make-keyspace-definition
([keyspace strategy-class replication-factor column-families]
(let [column-families (map (fn [{:keys [name comparator type validator]}]
(let [column-families (map (fn [{:keys [name comparator type validator comparator-alias]}]
(let [cf-def (if (nil? comparator)
(make-column-family keyspace name)
(make-column-family keyspace name comparator))]
(doto ^ThriftCfDef cf-def
(.setColumnType (column-type type))
(.setDefaultValidationClass (default-validation-class validator)))))
(.setDefaultValidationClass (default-validation-class validator)))
(if (not (nil? comparator-alias))
(.setComparatorTypeAlias cf-def comparator-alias))
cf-def
))
column-families)]

(HFactory/createKeyspaceDefinition keyspace
strategy-class
replication-factor
Expand Down
4 changes: 3 additions & 1 deletion test/clj_hector/test/core.clj
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@

(deftest-pending composite-serializer
(let [column-family "A"]
(with-test-keyspace keyspace [{:name column-family}]
(with-test-keyspace keyspace [{:name column-family
:comparator :composite
:comparator-alias "(AsciiType, AsciiType)"}]
(testing ":composite serializer"
(let [opts [:v-serializer :string
:n-serializer :composite]
Expand Down

0 comments on commit 3130161

Please sign in to comment.