Skip to content

Conversation

@lingrino
Copy link
Contributor

@lingrino lingrino commented May 5, 2022

Last minute I switched from using id to using key as the primary key because it makes more sense and maintains compatibility with the existing db. But turns out key is a reserved word in dynamodb so in order to use it we have to proxy it through an attribute name.

@lingrino lingrino requested a review from a team as a code owner May 5, 2022 15:53
Copy link

@wmiller112 wmiller112 left a comment

Choose a reason for hiding this comment

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

Interesting

@lingrino lingrino merged commit 57529ef into main May 5, 2022
@lingrino lingrino deleted the lingrino/key-as-attribute branch May 5, 2022 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants