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

Add secondary index scan #116

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

Conversation

C45tr0
Copy link
Contributor

@C45tr0 C45tr0 commented Oct 25, 2017

Updating #45. Let me know if this is the wrong way to handle this.

@coveralls
Copy link

coveralls commented Oct 25, 2017

Coverage Status

Coverage increased (+0.003%) to 98.664% when pulling cfd3a13 on C45tr0:add-secondary-index-scan into f0b023d on clarkie:master.

@jkav77
Copy link
Collaborator

jkav77 commented Jan 2, 2018

@C45tr0 I made a slight modification to the documentation by pulling the Indexes section out of the Query section since it now applies to Query and Scan. Is this good to go in your opinion?

@clarkie
Copy link
Owner

clarkie commented Jan 3, 2018

There were conflicts, I tried to resolve them but the tests are still failing. I'll have a look over coffee in the morning.

@cdhowie cdhowie added this to To Review in PRs Apr 9, 2018
@cdhowie cdhowie moved this from To Review to Needs updating in PRs Apr 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
PRs
  
Needs updating
Development

Successfully merging this pull request may close these issues.

None yet

5 participants