-
Notifications
You must be signed in to change notification settings - Fork 161
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate to aws-sdk-js-v3 document client #174
Migrate to aws-sdk-js-v3 document client #174
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 🙏🏻 Thank you so much for doing this! I hope this gets released soon!
Will this be merged anytime soon? |
It's been months since the PR's been created. I don't think it's getting merged anytime soon. :( |
Waiting for this one as well.. |
Is there any help needed with this work? |
Let's get this one merged v3 drastically reduces bundle size for serverless apps on lambda 🙌 |
@glcheetham technically you don't need to bundle v2 in your code since it is included in lambda function environment by default 🤷♂️ |
True, but worth knowing that it's an 'old' (6 months since release at time of writing - v2.1001.0) version, so make sure you peg your code to the same version to avoid any unexpected issues. |
This merge has been on hold because of the work on v0.4. This is now a high priority item, but will need a major refactor. Working with other collaborators to get this compatible with v0.4.x. |
@jeremydaly when is the timeline to get this bad boy in? Do you foresee it being compatible with the current dynamo tool box API or will it introduce breaking changes. Please note that DAX is not supported by aws s3 - so ideally dynamo toolbox should baked in the ability to switch easily between v2 and v3 |
Lambdas Node.js 18 runtime ships with sdk v3, would be nice if this would be merged soon https://docs.aws.amazon.com/lambda/latest/dg/lambda-runtimes.html |
At the moment I'm focusing on getting all the features/bug fixes in the issues section done, mostly because v3 is not backwards compatible, as far as I know. Let me know if I'm wrong, would love to add support for v3 |
V3 has some backwards compatibility interface but is not recommended to use it because it does not take advantage of the modularity of the new API |
dynamodb-onetable does support both V2 and V3 via a small abstraction layer. Maybe this idea could be useful here as well. |
@shishkin Thanks for calling this out! Anyone else looking for a good DynamoDB library with Deno, I can verify that dynamodb-onetable is working great with Deno 1.28 by way of these three imports and following the dynamodb-onetable documentation for setting up the connection, table, model, etc.
Looks like a great abstraction, hopefully dynamodb-toolbox will benefit from it too! |
Great idea :)
Adding support for Deno is a great idea and should be fairly easy to implement afaik, will add that as a card on our milestones board. |
Really keen to finally see this PR get over the line. Is there anything you need help with? |
I'm not sure whether to push it before closing the rest of the open issues or not. V1, will include SDK v3 support out of the box, along with a big re-write of most of the current code. At the same time, we could make a wrapper above the SDK v3 client, like mentioned in one of the above comments. If anyone wants to create a PR for this, go for it. |
What is the timeframe for v1? That might answer the question.. If it's less than 6 months, I'd say we wait. If it's more then we build a wrapper as discussed. |
There's no set date at the moment, |
Hi, any news with this PR ? |
Hey everyone, |
@naorpeled How close is this next update out of interest? |
I hope we'll be able to release it sometime during this weekend or next week 🙏 |
Hey everyone, |
Migrating from aws-sdk-js-v2 to aws-sdk-js-v3. (#155)
It uses DynamoDB Document Client v3.