clarkie / dynogels

DynamoDB data mapper for node.js. Originally forked from https://github.com/ryanfitz/vogels
Other
490 stars 110 forks source link

allow index scan #42

Closed stevenao closed 6 years ago

stevenao commented 7 years ago

the feature was added a few months ago in dynamodb. Can you add it to the scan module? currently, i manually add it to the scan prototype.

require('dynogels/lib/scan').prototype.usingIndex = function (name) { this.request.IndexName = name; return this; };

M1chaelTran commented 7 years ago

+1

jkav77 commented 6 years ago

Let's reopen this if there is some more activity here. Might not take too much effort to do a patch if the functionality is similar to the .usingIndex() method for .query().

thomasruiz commented 6 years ago

@dangerginger It'd be nice to have this. Currently we have dirty checks whether to use query or scan, leading to some complexity overall.