From c51556786fefe183e4e71ff46b1c898b1f60d12a Mon Sep 17 00:00:00 2001 From: Mark Paluch Date: Thu, 28 May 2020 13:54:07 +0200 Subject: [PATCH] DATAJDBC-546 - Skip property population if entity can be constructed entirely using a full constructor. --- .../data/jdbc/core/convert/BasicJdbcConverter.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java index 703ec5c55..49d94cf75 100644 --- a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java +++ b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java @@ -315,7 +315,6 @@ public class BasicJdbcConverter extends BasicRelationalConverter implements Jdbc private T populateProperties(T instance, @Nullable Object idValue) { PersistentPropertyAccessor propertyAccessor = getPropertyAccessor(entity, instance); - PreferredConstructor persistenceConstructor = entity.getPersistenceConstructor(); for (RelationalPersistentProperty property : entity) { @@ -460,7 +459,8 @@ public class BasicJdbcConverter extends BasicRelationalConverter implements Jdbc return readOrLoadProperty(idValue, property); }); - return populateProperties(instance, idValue); + + return entity.requiresPropertyPopulation() ? populateProperties(instance, idValue) : instance; } }