DATAREDIS-275 - Rename RedisServerCommands.bgWriteAof to bgReWriteAof.

Renamed method and placed @deprecated delegate-Methods in order to avoid breaking clients.

Original pull request: #44.

Author: Thomas Darimont

src/main/java/org/springframework/data/redis/connection/DefaultStringRedisConnection.java

@@ -42,6 +42,7 @@
  * @author Costin Leau
  * @author Jennifer Hickey
  * @author Christoph Strobl
+ * @author Thomas Darimont
  */
 public class DefaultStringRedisConnection implements StringRedisConnection {
 
@@ -135,8 +136,17 @@ public void bgSave() {
 		delegate.bgSave();
 	}
 
+	@Override
+	public void bgReWriteAof() {
+		delegate.bgReWriteAof();
+	}
+
+	/**
+	 * @deprecated As of 1.3, use {@link #bgReWriteAof}.
+	 */
+	@Deprecated
 	public void bgWriteAof() {
-		delegate.bgWriteAof();
+		bgReWriteAof();
 	}
 
 	public List<byte[]> bLPop(int timeout, byte[]... keys) {

src/main/java/org/springframework/data/redis/connection/RedisServerCommands.java

@@ -23,16 +23,27 @@
  * 
  * @author Costin Leau
  * @author Christoph Strobl
+ * @author Thomas Darimont
  */
 public interface RedisServerCommands {
 
 	/**
 	 * Start an {@literal Append Only File} rewrite process on server.
 	 * 
 	 * @see http://redis.io/commands/bgrewriteaof
+	 * @deprecated As of 1.3, use {@link #bgReWriteAof}.
 	 */
+	@Deprecated
 	void bgWriteAof();
 
+	/**
+	 * Start an {@literal Append Only File} rewrite process on server.
+	 * 
+	 * @see http://redis.io/commands/bgrewriteaof
+	 * @since 1.3
+	 */
+	void bgReWriteAof();
+
 	/**
 	 * Start background saving of db on server.
 	 * 

src/main/java/org/springframework/data/redis/connection/jedis/JedisConnection.java

@@ -42,7 +42,6 @@
 import org.springframework.data.redis.connection.convert.Converters;
 import org.springframework.data.redis.connection.convert.TransactionResultConverter;
 import org.springframework.util.Assert;
-import org.springframework.util.NumberUtils;
 import org.springframework.util.ObjectUtils;
 import org.springframework.util.ReflectionUtils;
 
@@ -466,7 +465,8 @@ public void bgSave() {
 		}
 	}
 
-	public void bgWriteAof() {
+	@Override
+	public void bgReWriteAof() {
 		try {
 			if (isPipelined()) {
 				pipeline(new JedisStatusResult(pipeline.bgrewriteaof()));
@@ -482,6 +482,14 @@ public void bgWriteAof() {
 		}
 	}
 
+	/**
+	 * @deprecated As of 1.3, use {@link #bgReWriteAof}.
+	 */
+	@Deprecated
+	public void bgWriteAof() {
+		bgReWriteAof();
+	}
+
 	public void save() {
 		try {
 			if (isPipelined()) {
@@ -2722,8 +2730,8 @@ public Long time() {
 		List<String> serverTimeInformation = this.jedis.time();
 
 		Assert.notEmpty(serverTimeInformation, "Received invalid result from server. Expected 2 items in collection.");
-		Assert.isTrue(serverTimeInformation.size() == 2, "Received invalid nr of arguments from redis server. Expected 2 received "
-				+ serverTimeInformation.size());
+		Assert.isTrue(serverTimeInformation.size() == 2,
+				"Received invalid nr of arguments from redis server. Expected 2 received " + serverTimeInformation.size());
 
 		return Converters.toTimeMillis(serverTimeInformation.get(0), serverTimeInformation.get(1));
 	}

src/main/java/org/springframework/data/redis/connection/jredis/JredisConnection.java

@@ -231,14 +231,22 @@ public void bgSave() {
 		}
 	}
 
-	public void bgWriteAof() {
+	public void bgReWriteAof() {
 		try {
 			jredis.bgrewriteaof();
 		} catch (Exception ex) {
 			throw convertJredisAccessException(ex);
 		}
 	}
 
+	/**
+	 * @deprecated As of 1.3, use {@link #bgReWriteAof}.
+	 */
+	@Deprecated
+	public void bgWriteAof() {
+		bgReWriteAof();
+	}
+
 	public void save() {
 		try {
 			jredis.save();

src/main/java/org/springframework/data/redis/connection/lettuce/LettuceConnection.java

@@ -46,7 +46,6 @@
 import org.springframework.data.redis.connection.convert.Converters;
 import org.springframework.data.redis.connection.convert.TransactionResultConverter;
 import org.springframework.util.Assert;
-import org.springframework.util.NumberUtils;
 import org.springframework.util.ObjectUtils;
 
 import com.lambdaworks.redis.RedisAsyncConnection;
@@ -480,7 +479,7 @@ public void bgSave() {
 		}
 	}
 
-	public void bgWriteAof() {
+	public void bgReWriteAof() {
 		try {
 			if (isPipelined()) {
 				pipeline(new LettuceStatusResult(getAsyncConnection().bgrewriteaof()));
@@ -496,6 +495,14 @@ public void bgWriteAof() {
 		}
 	}
 
+	/**
+	 * @deprecated As of 1.3, use {@link #bgReWriteAof}.
+	 */
+	@Deprecated
+	public void bgWriteAof() {
+		bgReWriteAof();
+	}
+
 	public void save() {
 		try {
 			if (isPipelined()) {
@@ -2776,7 +2783,7 @@ public Long time() {
 		 * support the time command natively.
 		 * see https://github.com/wg/lettuce/issues/19
 		 */
-		List<byte[]> result = (List<byte[]>)eval("return redis.call('TIME')".getBytes(),ReturnType.MULTI,0);
+		List<byte[]> result = (List<byte[]>) eval("return redis.call('TIME')".getBytes(), ReturnType.MULTI, 0);
 
 		Assert.notEmpty(result, "Received invalid result from server. Expected 2 items in collection.");
 		Assert.isTrue(result.size() == 2, "Received invalid nr of arguments from redis server. Expected 2 received "

src/main/java/org/springframework/data/redis/connection/srp/SrpConnection.java

@@ -60,6 +60,7 @@
  * @author Costin Leau
  * @author Jennifer Hickey
  * @author Christoph Strobl
+ * @author Thomas Darimont
  */
 public class SrpConnection implements RedisConnection {
 
@@ -363,7 +364,7 @@ public void bgSave() {
 		}
 	}
 
-	public void bgWriteAof() {
+	public void bgReWriteAof() {
 		try {
 			if (isPipelined()) {
 				pipeline(new SrpStatusResult(pipeline.bgrewriteaof()));
@@ -375,6 +376,14 @@ public void bgWriteAof() {
 		}
 	}
 
+	/**
+	 * @deprecated As of 1.3, use {@link #bgReWriteAof}.
+	 */
+	@Deprecated
+	public void bgWriteAof() {
+		bgReWriteAof();
+	}
+
 	public void save() {
 		try {
 			if (isPipelined()) {