BasicJdbcConverter supports Constructor Instantiate for classes with the @Transient property. [DATAJDBC-465]

[spring-projects-issues]

MyeongHyeonLee opened DATAJDBC-465 and commented

When loading data with the Constructor, an Exception is thrown if the @Transient property exists.

The @Transient property value can be set to null.

 

@AllArgsConstructor
public class DummyEntity {
    @Id
    private Long id;

    private String name;

    @Transient
    private String temp;
}

---

Affects: 1.1.3 (Moore SR3)

Referenced from: pull request #185

[spring-projects-issues]

Jens Schauder commented

I don't think properties marked with @Transient should be an argument to a constructor used for persistence.

 

Why don't you just provide a separate constructor without that parameter and mark it as @PersistenceConstructor?

[spring-projects-issues]

MyeongHyeonLee commented

Jens Schauder

I can define a Constructor using @PersistenceConstructor as you suggested.

This PR is suggested because I don't define Constructor separately but want to load Data with @AllArgsConstructor.

Since the @Transient field is not persistent, it makes sense to bind it as null

[spring-projects-issues]

Jens Schauder commented

I don't consider it a good idea to use a constructor although it is not matching the purpose and binding a @Transient field to anything would be a violation of the contract of @Transient.

Please use a separate constructor where you can set the @Transient field to anything matching your requirements

[spring-projects-issues]

MyeongHyeonLee commented

Jens Schauder

Thanks for the feedback. I will do that