clarkie / dynogels

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

Fix: runBatchGetItems fails to process UnprocessedKeys correctly #79

Closed RodH257 closed 7 years ago

RodH257 commented 7 years ago

Fixes https://github.com/clarkie/dynogels/issues/78

This is my first PR to this project, so please let me know if there's anything you'd like me to do differently. I've added a test to verify the behaviour, and have tested it on my own project and confirmed it fixes the bug I've encountered with large response (> 1MB).

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.002%) to 98.665% when pulling 3bfe10c9ad43630fba7c2bd46b0259bb43bdd93e on RodH257:fix_batchget into d327665b1ad475d1b2aba4ae4dd3a86db8b34685 on clarkie:master.