flybondi
feat: remove DynamodbWrapper client dependency
What the wrapper is doing for those cases is to be able to do an unlimited amount of request to the database taking care of the 25 request limit that the aws-sdk client has. So maybe it is a good idea to keep the wrapper client for those batch methods.
Thoughts?
:loop: Code coverage
| Filename | Statements | Branches | Functions | Lines |
|---|---|---|---|---|
| src/with-paginator.js | 100% | 95.83% | 100% | 100% |
| src/update.js | 100% | 100% | 100% | 100% |
| src/wrapper.js | 95% | 100% | 83.33% | 100% |
| src/table-name.js | 100% | 100% | 100% | 100% |
| src/return-values.js | 100% | 100% | 100% | 100% |
| src/map-merge-args.js | 100% | 100% | 100% | 100% |
| src/cast-array.js | 100% | 100% | 100% | 100% |
| src/generate-key.js | 100% | 100% | 100% | 100% |
| src/and-then.js | 100% | 100% | 100% | 100% |
| src/builders/append.js | 100% | 100% | 100% | 100% |
| src/builders/catcon.js | 100% | 100% | 100% | 100% |
| src/builders/put.js | 100% | 100% | 100% | 100% |
| src/builders/update-expression.js | 100% | 100% | 100% | 100% |
| src/batch-get-item.js | 100% | 100% | 100% | 100% |
| src/get.js | 100% | 100% | 100% | 100% |
| src/query.js | 100% | 75% | 100% | 100% |
| src/remove.js | 100% | 100% | 100% | 100% |
| src/batch-write-item.js | 100% | 100% | 100% | 100% |
| src/generate-item.js | 100% | 100% | 100% | 100% |
| src/insert.js | 100% | 100% | 100% | 100% |
| src/builders/remove.js | 100% | 100% | 100% | 100% |
| src/get-all.js | 100% | 60% | 100% | 100% |
| src/count.js | 100% | 100% | 100% | 100% |
| src/describe-table.js | 100% | 100% | 100% | 100% |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
What the wrapper is doing for those cases is to be able to do an unlimited amount of request to the database taking care of the 25 request limit that the aws-sdk client has. So maybe it is a good idea to keep the wrapper client for those batch methods.
Thoughts?
@lfantone we may need to drop the dependency because it's blocking the upgrade to aws-sdk v3. Maybe we can bring the implementation for the batch methods to flynamo itself?
