Skip to content

Add secondary index scan#116

Open
C45tr0 wants to merge 7 commits intoclarkie:masterfrom
C45tr0:add-secondary-index-scan
Open

Add secondary index scan#116
C45tr0 wants to merge 7 commits intoclarkie:masterfrom
C45tr0:add-secondary-index-scan

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.

@C45tr0 C45tr0 force-pushed the add-secondary-index-scan branch from d1f90b2 to 78dd40f Compare June 16, 2018 06:52
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.

5 participants