DATAREDIS-988 - Polishing.

Original Pull Request: #453

Author: christophstrobl

src/main/java/org/springframework/data/redis/core/RedisConnectionUtils.java

@@ -187,7 +187,7 @@ private static RedisConnection createConnectionProxy(RedisConnection connection,
 	 *
 	 * @param conn the Redis connection to close.
 	 * @param factory the Redis factory that the connection was created with.
-	 * @deprecated since 2.1.9, use {@link #releaseConnection(RedisConnection, RedisConnectionFactory, boolean)}
+	 * @deprecated since 2.1.10, use {@link #releaseConnection(RedisConnection, RedisConnectionFactory, boolean)} instead.
 	 */
 	@Deprecated
 	public static void releaseConnection(@Nullable RedisConnection conn, RedisConnectionFactory factory) {
@@ -200,11 +200,11 @@ public static void releaseConnection(@Nullable RedisConnection conn, RedisConnec
 	 *
 	 * @param conn the Redis connection to close.
 	 * @param factory the Redis factory that the connection was created with.
-	 * @param enableTransactionSupport whether transaction support is enabled.
+	 * @param transactionSupport whether transaction support is enabled.
 	 * @since 2.1.9
 	 */
 	public static void releaseConnection(@Nullable RedisConnection conn, RedisConnectionFactory factory,
-			boolean enableTransactionSupport) {
+			boolean transactionSupport) {
 
 		if (conn == null) {
 			return;
@@ -223,23 +223,23 @@ public static void releaseConnection(@Nullable RedisConnection conn, RedisConnec
 
 			// release transactional/read-only and non-transactional/non-bound connections.
 			// transactional connections for read-only transactions get no synchronizer registered
-			if (enableTransactionSupport && TransactionSynchronizationManager.isCurrentTransactionReadOnly()) {
+			if (transactionSupport && TransactionSynchronizationManager.isCurrentTransactionReadOnly()) {
 				if (log.isDebugEnabled()) {
-					log.debug("Unbinding Redis Connection");
+					log.debug("Unbinding Redis Connection.");
 				}
 				unbindConnection(factory);
 			} else {
 
 				// Not participating in transaction management.
 				// Connection could have been attached via session callback.
 				if (log.isDebugEnabled()) {
-					log.debug("Leaving bound Redis Connection attached");
+					log.debug("Leaving bound Redis Connection attached.");
 				}
 			}
 
 		} else {
 			if (log.isDebugEnabled()) {
-				log.debug("Closing Redis Connection");
+				log.debug("Closing Redis Connection.");
 			}
 			conn.close();
 		}

src/test/java/org/springframework/data/redis/core/RedisTemplateUnitTests.java

@@ -111,6 +111,7 @@ public void executeSessionShouldReuseConnection() {
 			@Nullable
 			@Override
 			public <K, V> Object execute(RedisOperations<K, V> operations) throws DataAccessException {
+
 				template.multi();
 				template.multi();
 				return null;
@@ -129,6 +130,7 @@ public void executeSessionInTransactionShouldReuseConnection() {
 		template.execute(new SessionCallback<Object>() {
 			@Override
 			public <K, V> Object execute(RedisOperations<K, V> operations) throws DataAccessException {
+
 				template.multi();
 				template.multi();
 				return null;
@@ -146,8 +148,10 @@ public void transactionAwareTemplateShouldReleaseConnection() {
 		TransactionSynchronizationManager.setCurrentTransactionReadOnly(true);
 
 		template.execute(new SessionCallback<Object>() {
+
 			@Override
 			public <K, V> Object execute(RedisOperations<K, V> operations) throws DataAccessException {
+
 				template.multi();
 				template.multi();
 				return null;