Query.js: Added `limit` and `cursor` #23

Closed
wants to merge 18 commits into
from

Conversation

Projects
None yet
2 participants

I created a first version to solve your To do: Add method to specify Limit and ExclusiveStartKey
Can you have a took at it and tell me your opinion?

Hello Jed,

i also added a fix for correct handling of Action:"Delete" on Dynamo sets

@mpneuried mpneuried - renamed `cursor` to `startAt`
- added `limit` and `startAt` to `Scan`
b6e5e13

Next change ;-)
I added the limit and (now) startAt methods to Scan.js, too.

Owner

jed commented May 24, 2012

thanks @mpneuried! i'm headed on vacation for the next week, but will check on these when i get back, okay?

Hi @jed. I wish you a relaxing vacation. Don't hurry ;-)

I'm in heavy development for my abstraction. So if i find some bugs or do some changes to your module, i will update this pull request.

As i promised ... here comes the next change. I added options + the option returnvalues to the item.destroy( [options], callback ). Now it's possible to define the Returnvalues key for the AWS DeleteItem request.

Next update. I found a bug in Batch.js. The check for UnprocessedKeys was true on a empty object. So the module tried a second empty call which produced an error.

jed closed this Mar 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment