Simplified EntityRowMapper.

Removes an unused field and moves operations into  constructor, to make the mapRow method simpler.

Closes #1974

Author: schauder

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/EntityRowMapper.java

@@ -21,8 +21,8 @@
 import org.springframework.data.relational.core.mapping.AggregatePath;
 import org.springframework.data.relational.core.mapping.RelationalPersistentEntity;
 import org.springframework.data.relational.domain.RowDocument;
+import org.springframework.data.util.TypeInformation;
 import org.springframework.jdbc.core.RowMapper;
-import org.springframework.lang.Nullable;
 
 /**
  * Maps a {@link ResultSet} to an entity of type {@code T}, including entities referenced. This {@link RowMapper} might
@@ -37,36 +37,32 @@
  */
 public class EntityRowMapper<T> implements RowMapper<T> {
 
-	private final RelationalPersistentEntity<T> entity;
-	private final AggregatePath path;
+	private final TypeInformation<T> typeInformation;
 	private final JdbcConverter converter;
-	private final @Nullable Identifier identifier;
+	private final Identifier identifier;
 
-	@SuppressWarnings("unchecked")
-	public EntityRowMapper(AggregatePath path, JdbcConverter converter, Identifier identifier) {
+	private EntityRowMapper(TypeInformation<T> typeInformation, JdbcConverter converter, Identifier identifier) {
 
-		this.entity = (RelationalPersistentEntity<T>) path.getLeafEntity();
-		this.path = path;
+		this.typeInformation = typeInformation;
 		this.converter = converter;
 		this.identifier = identifier;
 	}
 
-	public EntityRowMapper(RelationalPersistentEntity<T> entity, JdbcConverter converter) {
+	@SuppressWarnings("unchecked")
+	public EntityRowMapper(AggregatePath path, JdbcConverter converter, Identifier identifier) {
+		this(((RelationalPersistentEntity<T>) path.getRequiredLeafEntity()).getTypeInformation(), converter, identifier);
+	}
 
-		this.entity = entity;
-		this.path = null;
-		this.converter = converter;
-		this.identifier = null;
+	public EntityRowMapper(RelationalPersistentEntity<T> entity, JdbcConverter converter) {
+		this(entity.getTypeInformation(), converter, Identifier.empty());
 	}
 
 	@Override
 	public T mapRow(ResultSet resultSet, int rowNumber) throws SQLException {
 
 		RowDocument document = RowDocumentResultSetExtractor.toRowDocument(resultSet);
 
-		return identifier == null //
-				? converter.readAndResolve(entity.getTypeInformation(), document, Identifier.empty()) //
-				: converter.readAndResolve(entity.getTypeInformation(), document, identifier);
+		return converter.readAndResolve(typeInformation, document, identifier);
 	}
 
 }