From 312085aa2e7c3f30ba03d0d234e6aa989e80d10d 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 e4d06f0d..36c50c58 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 @@ -390,7 +390,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) { @@ -534,7 +533,8 @@ public class BasicJdbcConverter extends BasicRelationalConverter implements Jdbc RelationalPersistentProperty property = entity.getRequiredPersistentProperty(parameterName); return readOrLoadProperty(idValue, property); }); - return populateProperties(instance, idValue); + + return entity.requiresPropertyPopulation() ? populateProperties(instance, idValue) : instance; } }