search

pingles / clj-hector

Simple Cassandra client for Clojure
42 stars 19 forks source link

Adds column-metadata option to Column Family at creation. #22

Closed alanpeabody closed 12 years ago

alanpeabody commented 12 years ago

The tests could use some more flushing out: there are no tests actually exercising the secondary indexes.

Any feedback would be great!

Ref #21

nickmbailey commented 12 years ago

This looks good to me. I noticed some mismatch in how we do some conversions which inspired me to create #23 but I don't think there is anything specifically to do here.

Are you also planning on adding query capabilities for indexed columns as part of this ticket. If so, we can just include that as part of this pull request. If not we can close this pull request/issue and open a different issue for adding the ability to query the index.

alanpeabody commented 12 years ago

Frankly I was just planning on using CQL for querying secondary indexes. I have not even investigated how to do so with Hector.

I may be able to take a look at hector secondary indexes either on Friday or this weekend. It is at your discretion weather you would like that as part of this pull request or not.