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

Fix Inefficent Range Filtering Error #39

Merged
merged 1 commit into from
May 7, 2015

Conversation

sethdenner
Copy link
Contributor

There was an error in how the backend determines wether or not a range query can be evaluated efficiently by Cassandra. It was allowing attempts of efficent filtering on non-clustering columns.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 51.26% when pulling a14eb54 on sethdenner:release-0.1.2 into f8ea55b on Knotis:master.

There was an error in how the backend determines wether or not a range query can be evaluated efficiently by Cassandra. It was allowing attempts of efficent filtering on non-clustering columns.

Had some errors in how I was handling deleting and counting methods. I thought the queries were build at that point in code execution. I was mistaken but now the queries are build correctly and the results are as expected.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.62%) to 50.65% when pulling 45538f4 on sethdenner:release-0.1.2 into f8ea55b on Knotis:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.62%) to 50.65% when pulling 45538f4 on sethdenner:release-0.1.2 into f8ea55b on Knotis:master.

sethdenner added a commit that referenced this pull request May 7, 2015
Fix Inefficent Range Filtering Error
@sethdenner sethdenner merged commit 5fa7e66 into Knotis:master May 7, 2015
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.

None yet

2 participants