[GitHub] cassandra pull request #85: sasi: AND optimizations

2016-12-28 Thread iksaif
Github user iksaif closed the pull request at:

https://github.com/apache/cassandra/pull/85


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cassandra issue #85: sasi: AND optimizations

2016-11-22 Thread iksaif
Github user iksaif commented on the issue:

https://github.com/apache/cassandra/pull/85
  
Done, I'm not yet very familiar with the workflow.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cassandra pull request #85: sasi: AND optimizations

2016-11-22 Thread iksaif
GitHub user iksaif opened a pull request:

https://github.com/apache/cassandra/pull/85

sasi: AND optimizations

when op is AND:
- do not use indexes that return way more results that the others
- do not use more indexes if one of them returns no results
- try to use something smarter to get the primary index (which is
  important because it limits the searches done on other indexes)

Cosmetic changes such as changing logger.debug() to logger.trace() will be 
done later, I'd like a first opinion on the concept.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/iksaif/cassandra cassandra-3.X

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cassandra/pull/85.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #85


commit bc54f7ae176e9314190c49d1780fb87e26b62728
Author: Corentin Chary <c.ch...@criteo.com>
Date:   2016-11-21T16:04:42Z

sasi: AND optimizations

when op is AND:
- do not use indexes that return way more results that the others
- do not use more indexes if one of them returns no results
- try to use something smarter to get the primary index (which is
  important because it limits the searches done on other indexes)




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---