Add support for CQL queries. #19

Merged
merged 2 commits into from Mar 5, 2012

Projects

None yet

2 participants

@mstump
Contributor
mstump commented Feb 28, 2012

I've added support for fetching rows via CQL. The one odd behavior is that hector is adding a column KEY to the result so if the column values are {a 1} with a row key of k, you get back {KEY k a 1}. I can't filter it out because if the user has a column KEY then I'll end up dropping their data. What do you want to do?

@nickmbailey nickmbailey was assigned Mar 2, 2012
@nickmbailey
Collaborator

Looks like hector actually has a 'setSuppressKeyInColumns' method on the CqlQuery object. That should remove the row key from the column list. I don't see a reason to not just always use that setting for cql queries form clj hector.

Assuming you agree, want to update to use that?

@mstump
Contributor
mstump commented Mar 2, 2012

Thanks for finding that, I'll update the pull request tonight.

@mstump
Contributor
mstump commented Mar 5, 2012

It's ready for your review. Sorry for the delay.

@nickmbailey nickmbailey merged commit 9e798c1 into pingles:master Mar 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment