Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Issues/2576 dynamo db entry conversion #3735

Conversation

irina-herciu
Copy link
Contributor

Allow to set DynamoDBEntryConversion per table

Description

Update DynamoDBTableAttribute in order to allow ConversionSchema to be set at table level.

[DynamoDBTable("HashTable", false, ConversionSchema.V2)]

The ConversionSchema apply as follows:

  • OperationConfig Conversion Schema
  • Table Conversion Schema
  • Context Conversion Schema

Motivation and Context

#2576

Testing

Integration test added

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have read the README document
  • I have added tests to cover my changes
  • All new and existing tests passed

License

  • I confirm that this pull request can be released under the Apache 2 license

add change file

Allow to set DynamoDBEntryConversion per table
@irina-herciu irina-herciu force-pushed the issues/2576_DynamoDBEntryConversion branch from 15db54c to c039186 Compare April 1, 2025 17:28
@normj normj added the v4 label Apr 7, 2025
Copy link
Member

@normj normj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dry run DRY_RUN-98864961-5f50-49fe-91a4-d9b735b91842 was successful

@dscpinheiro dscpinheiro merged commit 19697c2 into aws:v4-development Apr 8, 2025
@normj
Copy link
Member

normj commented Apr 9, 2025

This PR was released as part of V4 preview 13.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants