derjust / spring-data-dynamodb

This module deals with enhanced support for a data access layer built on AWS DynamoDB.
https://derjust.github.io/spring-data-dynamodb/
Apache License 2.0
403 stars 141 forks source link

Upgrading to 5.1.0 results in the error Bean property 'dynamoDBMapperConfig' is not writable or has an invalid setter method. #235

Closed davidmelia closed 5 years ago

davidmelia commented 5 years ago

Hi,

I have recreated this error in https://github.com/davidmelia/spring-data-dynamodb-bug (run the test DynamoDbBugTests)

In a nutshell, after upgrading to 5.1.0 I get the error below:

Caused by: org.springframework.beans.NotWritablePropertyException: Invalid property 'dynamoDBMapperConfig' of bean class [org.socialsignin.spring.data.dynamodb.repository.support.DynamoDBRepositoryFactoryBean]: Bean property 'dynamoDBMapperConfig' is not writable or has an invalid setter method. Does the parameter type of the setter match the return type of the getter?
    at org.springframework.beans.BeanWrapperImpl.createNotWritablePropertyException(BeanWrapperImpl.java:243) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    at org.springframework.beans.AbstractNestablePropertyAccessor.processLocalProperty(AbstractNestablePropertyAccessor.java:426) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    at org.springframework.beans.AbstractNestablePropertyAccessor.setPropertyValue(AbstractNestablePropertyAccessor.java:278) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    at org.springframework.beans.AbstractNestablePropertyAccessor.setPropertyValue(AbstractNestablePropertyAccessor.java:266) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    at org.springframework.beans.AbstractPropertyAccessor.setPropertyValues(AbstractPropertyAccessor.java:97) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    at org.springframework.beans.AbstractPropertyAccessor.setPropertyValues(AbstractPropertyAccessor.java:77) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.applyPropertyValues(AbstractAutowireCapableBeanFactory.java:1681) ~[spring-beans-5.1.4.RELEASE.jar:5.1.4.RELEASE]
    ... 55 common frames omitted
jakobgalbavy commented 5 years ago

I guess, this is related to #230

shexbeer commented 5 years ago

Get the same error when updating to 5.1.0 and spring boot 2.1.3 Is this really related to #230 because it does not occur on Tests. It occurs on application run

derjust commented 5 years ago

Probably related, yes. This one is fixed in master and will be part of 5.1.1

ghsatpute commented 5 years ago

Hi @derjust any expected date of release of 5.1.1?

allan-todd commented 5 years ago

Hi @derjust , I'd also be keen to know when 5.1.1 will be released as we're a bit stuck at the moment. We had to upgrade from 5.0.4 to fix https://github.com/derjust/spring-data-dynamodb/issues/218 but can't use 5.1.0 owing to this issue.

Thanks

shexbeer commented 5 years ago

Regarding #230 and #237 it would be good to have a fix here!

lwis commented 5 years ago

Any news on a 5.1.1 release?

psabharwal123 commented 5 years ago

@derjust ^

mpataki commented 5 years ago

I'm starting a new project on boot 2.1.8 and 5.1.0 and hitting this issue. :pray:

lmeadows commented 5 years ago

@derjust Is 5.1.1 ready to go? I am considering making my own maven package based on the current state of master in this repo and vendoring it into my app so that I can have this crucial fix. My main concern is that the 5.1.1 release is being held up by quality problems. Is it ready for production?

boostchicken commented 5 years ago

@lmeadows

https://github.com/boostchicken/spring-data-dynamodb

api  group: 'io.github.boostchicken', name: 'spring-data-dynamodb', version: '5.2.0-SNAPSHOT'

on OSSRH, give it go and let me know if it works and I'll release soon. Has Spring Boot 2.2 support

snakeru commented 5 years ago

Thank you, @boostchicken

My test compiles and runs now!

boostchicken commented 5 years ago

api group: 'io.github.boostchicken', name: 'spring-data-dynamodb', version: '5.2.1' On maven central now

dlaroue4 commented 5 years ago

@boostchicken Thank you!

algimantasligeikis commented 4 years ago

'io.github.boostchicken

Thank you very much, @boostchicken you are wonderful person :)