-
Notifications
You must be signed in to change notification settings - Fork 67
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
FindHasMany is limited to 25 (default) hits #81
Comments
I am not aware there is a limit. Will investigate. On https://bloggr.exmer.com/ it will return more than 25 Posts. See https://bloggr.exmer.com/authors/D8F61FBC-A9C4-0318-851B-228507CDC032 after this record is loaded. https://github.com/broerse/ember-cli-blog/blob/master/app/models/author.js |
Hmm... I wonder if this is only when running against CouchDB: http://docs.couchdb.org/en/2.1.0/api/database/find.html?highlight=limit#db-find Perhaps PouchDB does not have a default limit? My scenario is that I use Ember-Pouch running against a PouchDB syncing to CouchDB on the iPad, and in the browser running straight against the same CouchDB. |
We run everything against Cloudant or CouchDB 1.6 and it does not have this limit. I see this limit in your CouchDB 2.1.0 doc but it does not state how to disable it. See also https://stackoverflow.com/questions/45632794/how-to-fetch-unlimited-documents-using-mango-query-couchdb-without-providing-li |
@nolanlawson Is this fixed/addressed in the newest pouch-find. |
@afinne Cloudant has this limit but we never saw it because if you use find on the local pouch it does not have this limit. So it is only the backend that has this limit. We stil need to fix this somewhere. I think this module is perhaps the correct place to fix this and not in |
I've had this exact same issue using ember-pouch with CouchDB and went through hours of exploration of messing with adapters and scouring through documentation to eventually come to a simple workaround. As mentioned above, CouchDB 2.1.0 added the default limit. With ember-pouch v5.0.0 you can now set limit on query, but this didn't affect findHasMany for me. By searching through the Therefore if you don't want a limit, you can just set a high limit under |
@rhysdavies1994 I did't know that. Not sure this is what we shout advise but until we fix this for findHasMany it is a good workaround. |
For a bit of clarification on @rhysdavies1994’s workaround: you can enlarge the find default in Fauxton by:
|
@muziejus solution working like a charm. Here's another pic: |
When using ember-pouch (or relational-pouch), when following a
hasMany
link, it doesn't set an unlimited limit, so I only get the first 25 childrenFor instance when listing all managers, that have a
managerFor
relation to technicians (inverse viamanager
) results in a_find
wheremanager
= the id. However, since no limit is included, I only ever get the 25 first technicians for the manager.The code in question is: https://github.com/pouchdb-community/relational-pouch/blob/master/lib/index.js#L425-L438
The text was updated successfully, but these errors were encountered: