-
Notifications
You must be signed in to change notification settings - Fork 141
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
TableNameOverride Throwing Error #146
Comments
I was able to coax a stacktrace out:
|
Temporary work arounds are appreciated as well. |
As this issue appears in multiple flavors and scenarios it's on top of my stack right now. |
…equired fields The old documentaiton at https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime (Same as https://git.io/DynamoDBMapperConfig) told people to start with an empty Builder. This causes issues down the line as required fields (TypeConverterFactory & ConversionSchema) where not properly initialized
…equired fields The old documentaiton at https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime (Same as https://git.io/DynamoDBMapperConfig) told people to start with an empty Builder. This causes issues down the line as required fields (TypeConverterFactory & ConversionSchema) where not properly initialized
@adenix I think right now you have two options: a) I updated the wiki article. If you follow the changed approach on how to create the b) OR if you feel lucky you may want to try |
@derjust My service isn't going live for another couple of weeks so Ill go ahead and try out option B and fallback to A if that does work. Thanks for your quick work and I'll make sure to update you with the result. |
Snapshot solved issue and had no breaking changes for my other operations. One note: I took a look at option 'a' and I was already using |
…ed fields The old documentaiton at https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime (Same as https://git.io/DynamoDBMapperConfig) told people to start with an empty Builder. This causes issues down the line as required fields (TypeConverterFactory & ConversionSchema) where not properly initialized Also updated https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime
…ed fields The old documentaiton at https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime (Same as https://git.io/DynamoDBMapperConfig) told people to start with an empty Builder. This causes issues down the line as required fields (TypeConverterFactory & ConversionSchema) where not properly initialized Also updated https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime
…ed fields The old documentaiton at https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime (Same as https://git.io/DynamoDBMapperConfig) told people to start with an empty Builder. This causes issues down the line as required fields (TypeConverterFactory & ConversionSchema) where not properly initialized Also updated https://github.com/derjust/spring-data-dynamodb/wiki/Alter-table-name-during-runtime
Updated Wiki and fixed in upcoming |
Issue #146 - Initialize DynamoDBMapperConfigs that are missing required fields
Any dates for 5.0.3 release? Facing same issue.(Discussed workaround isn't solving with 5.0.2) |
Expected Behavior
Modifying configuration to use
dynamoDBMapperConfig
bean with atableNameOverride
should simply change the DynamoDB table name that we are accessingActual Behavior
Modifying configuration causes
Circular view path [error]
Steps to Reproduce the Problem
Specifications
Spring Data DynamoDB Version:
5.0.2
Spring Data Version:
AWS SDK Version:
1.11.220
Java Version:
1.8.0_162
Spring Boot Version:
2.0.0.RELEASE
The text was updated successfully, but these errors were encountered: