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

Support 'all/in/and... in...' queries for composite primary keys, > 100 records #15

Open
darrenklein opened this issue Feb 22, 2019 · 0 comments
Labels
to-do Features/tasks that we'd like to add to the adapter.

Comments

@darrenklein
Copy link
Contributor

At the moment, using a query to search for more than 100 records on a composite primary key will fail, when using an 'all/in/and... in...' queries, such as

from bp in BookPage, where: bp.id in ^ids and bp.page_num in ^pages

Queries where the second condition is a variable (as above) will fail for different reasons than a hard-coded list - in the former, queries for more than 50 records will actually fail because both lists will be merged and seem like a request for more than 100 records; in the latter, the current method of breaking the query into chunks of 100 items will only affect the hash key, and so on the next iteration the values of the two indexes will fall out of sync.

@darrenklein darrenklein added the to-do Features/tasks that we'd like to add to the adapter. label Feb 22, 2019
@darrenklein darrenklein changed the title TO-DO: support 'all/in/and... in...' queries for composite primary keys, > 100 records Support 'all/in/and... in...' queries for composite primary keys, > 100 records Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to-do Features/tasks that we'd like to add to the adapter.
Projects
None yet
Development

No branches or pull requests

1 participant