Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generalize handling WHERE IN clauses #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

annismckenzie
Copy link

This cleans up the changes I made in #3. It pushes clauses for WHERE col IN (?) statements into the where clause handler, thereby drops the string value constraint and then lets gocql fill in the values correctly interpolated and escaped.

If you want I can get started on a few basic tests – however, I've been using WHERE IN extensively in our codebase and the changes are sufficiently easy to understand.

It’s now using the same mechanism as `Where` and isn’t constrained to
string values anymore. Quoting and escaping is deferred to gocql, making
the functionality more generally usable.
@annismckenzie annismckenzie changed the title Generalize WhereIn Generalize handling WHERE IN clauses Nov 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant