Fix #99. Break up big queries into little ones

coresmart / persistencejs

persistence.js is an asynchronous Javascript database mapper library. You can use it in the browser, as well on the server (and you can share data models between them).

http://persistencejs.org

1.73k stars 240 forks source link

Fix #99. Break up big queries into little ones #100

Closed pixelcort closed 11 years ago

pixelcort commented 11 years ago

Also does the filtering out in code in one of them.

This fixes #99 where if there are too many ids, these queries can fail to work.

pixelcort commented 11 years ago

Whoops, looks like there's a typo in this fix. Hold off on integrating it; I'll update this once I commit the fix into this PR.

pixelcort commented 11 years ago

Okay this looks good now and is probably ready for pulling in.

There's a second O(n^2) performance problem with syncing locally deleted records, but I think that's a separate issue as it appears to also be there before these fixes.

zefhemel commented 11 years ago

Thanks!