From 46f65149b89f4b519a8eee44317c3897c01f6107 Mon Sep 17 00:00:00 2001
From: Jens Schauder <jschauder@pivotal.io>
Date: Tue, 20 Mar 2018 10:36:39 +0100
Subject: [PATCH] DATAJDBC-186 - Polishing.

JavaDoc.
---
 .../data/jdbc/core/EntityRowMapper.java               | 11 +++++++++++
 .../jdbc/core/DefaultDataAccessStrategyUnitTests.java |  2 ++
 2 files changed, 13 insertions(+)

diff --git a/src/main/java/org/springframework/data/jdbc/core/EntityRowMapper.java b/src/main/java/org/springframework/data/jdbc/core/EntityRowMapper.java
index 89dae3e2..67f3b435 100644
--- a/src/main/java/org/springframework/data/jdbc/core/EntityRowMapper.java
+++ b/src/main/java/org/springframework/data/jdbc/core/EntityRowMapper.java
@@ -101,6 +101,16 @@ public class EntityRowMapper<T> implements RowMapper<T> {
 		return instantiator.createInstance(entity, new ResultSetParameterValueProvider(rs, entity, conversions, ""));
 	}
 
+
+	/**
+	 * Read a single value or a complete Entity from the {@link ResultSet} passed as an argument.
+	 *
+	 * @param resultSet the {@link ResultSet} to extract the value from. Must not be {@code null}.
+	 * @param property the {@link JdbcPersistentProperty} for which the value is intended. Must not be {@code null}.
+	 * @param prefix to be used for all column names accessed by this method. Must not be {@code null}.
+	 *
+	 * @return the value read from the {@link ResultSet}. May be {@code null}.
+	 */
 	private Object readFrom(ResultSet resultSet, JdbcPersistentProperty property, String prefix) {
 
 		try {
@@ -110,6 +120,7 @@ public class EntityRowMapper<T> implements RowMapper<T> {
 			}
 
 			return resultSet.getObject(prefix + property.getColumnName());
+
 		} catch (SQLException o_O) {
 			throw new MappingException(String.format("Could not read property %s from result set!", property), o_O);
 		}
diff --git a/src/test/java/org/springframework/data/jdbc/core/DefaultDataAccessStrategyUnitTests.java b/src/test/java/org/springframework/data/jdbc/core/DefaultDataAccessStrategyUnitTests.java
index 6c3ae9f2..91457a12 100644
--- a/src/test/java/org/springframework/data/jdbc/core/DefaultDataAccessStrategyUnitTests.java
+++ b/src/test/java/org/springframework/data/jdbc/core/DefaultDataAccessStrategyUnitTests.java
@@ -32,6 +32,8 @@ import org.springframework.jdbc.core.namedparam.SqlParameterSource;
 import org.springframework.jdbc.support.KeyHolder;
 
 /**
+ * Unit tests for {@link DefaultDataAccessStrategy}.
+ *
  * @author Jens Schauder
  */
 public class DefaultDataAccessStrategyUnitTests {