Skip to content
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

DynamodDB Type converters #1664

Draft
wants to merge 44 commits into
base: 5.0.x
Choose a base branch
from
Draft

DynamodDB Type converters #1664

wants to merge 44 commits into from

Conversation

sdelamo
Copy link
Contributor

@sdelamo sdelamo commented Mar 28, 2023

This PR contains TypeConverter from and to AttributeValue for:

It is the first step towards improving the integration of Micronaut Framework with Amazon DynamoDB.

In a future PR we could leverage these converters to provide conversion between a introspected class to a Map<String, AttributeValue>

@sdelamo
Copy link
Contributor Author

sdelamo commented Mar 28, 2023

@radovanradic let me know if this conflicts with your spike here: micronaut-projects/micronaut-data@master...data-aws-dynamodb

@radovanradic
Copy link

@radovanradic let me know if this conflicts with your spike here: micronaut-projects/[email protected]

No, this could be useful for that work in data. I just started with connectivity and few methods in repository implemented and switched to something else so I will definitely be able to use this later.

@sdelamo sdelamo force-pushed the dynamodb-converters branch from 0ee9153 to 41daa1a Compare March 29, 2023 15:31
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@scprek
Copy link
Contributor

scprek commented Nov 21, 2023

Moving discussion here from micronaut-projects/micronaut-core#10137

Any plans for an EnhancedDynamoDbClient in Micronaut? https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/dynamodb-enhanced-client.html

We just do it in a library so the Beans are available for use. I think we had to do something custom with @secondary constructors to make it work with micronaut test resources as well. I can follow up on this

@scprek
Copy link
Contributor

scprek commented Apr 30, 2024

Moving discussion here from micronaut-projects/micronaut-core#10137

Any plans for an EnhancedDynamoDbClient in Micronaut? https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/dynamodb-enhanced-client.html

We just do it in a library so the Beans are available for use. I think we had to do something custom with @secondary constructors to make it work with micronaut test resources as well. I can follow up on this

I see this is the current solution, excluding doing yourself, https://micronaut-projects.github.io/micronaut-aws/latest/guide/#thirdPartyLibraries

@musketyr
Copy link
Contributor

musketyr commented May 2, 2024

thanks for mentioning @scprek. The v2 implementation works nicely with EnhancedDynamoDbClient and you can use Integration Testing library to start up a Localstack under the hood without any additional configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

4 participants