Add support for CONFIG REWRITE.

Signed-off-by: Manousos Mathioudakis <[email protected]>

Closes #1992
Original pull request: #2012.

Author: Manousos Mathioudakis

Diff for: pom.xml

@@ -1,5 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" standalone="no"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
 
 	<modelVersion>4.0.0</modelVersion>
 

Diff for: src/main/java/org/springframework/data/redis/connection/DefaultStringRedisConnection.java

@@ -3731,6 +3731,11 @@ public void migrate(byte[] key, RedisNode target, int dbIndex, @Nullable Migrate
 		delegate.migrate(key, target, dbIndex, option, timeout);
 	}
 
+	@Override
+	public void rewriteConfig() {
+		delegate.rewriteConfig();
+	}
+
 	/*
 	 * (non-Javadoc)
 	 * @see org.springframework.data.redis.connection.StringRedisConnection#xAck(java.lang.String, java.lang.String, RecordId[])

Diff for: src/main/java/org/springframework/data/redis/connection/RedisServerCommands.java

@@ -280,4 +280,9 @@ default Long time() {
 	 * @see <a href="https://redis.io/commands/migrate">Redis Documentation: MIGRATE</a>
 	 */
 	void migrate(byte[] key, RedisNode target, int dbIndex, @Nullable MigrateOption option, long timeout);
+
+	/**
+	 * Rewrites the redis.conf file.
+	 */
+	void rewriteConfig();
 }

Diff for: src/main/java/org/springframework/data/redis/connection/jedis/JedisClusterConnection.java

@@ -869,6 +869,11 @@ public RedisSentinelConnection getSentinelConnection() {
 		throw new UnsupportedOperationException("Sentinel is currently not supported for JedisClusterConnection.");
 	}
 
+	@Override
+	public void rewriteConfig() {
+		serverCommands().rewriteConfig();
+	}
+
 	/**
 	 * {@link Jedis} specific {@link ClusterCommandCallback}.
 	 *

Diff for: src/main/java/org/springframework/data/redis/connection/jedis/JedisClusterServerCommands.java

@@ -520,6 +520,12 @@ public void migrate(byte[] key, RedisNode target, int dbIndex, @Nullable Migrate
 				node);
 	}
 
+	@Override
+	public void rewriteConfig() {
+		connection.getClusterCommandExecutor()
+				.executeCommandOnAllNodes((JedisClusterCommandCallback<String>) client -> client.configRewrite());
+	}
+
 	private Long convertListOfStringToTime(List<String> serverTimeInformation, TimeUnit timeUnit) {
 
 		Assert.notEmpty(serverTimeInformation, "Received invalid result from server. Expected 2 items in collection.");

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

@@ -806,4 +806,8 @@ private void doWithJedis(Consumer<Jedis> callback) {
 		}
 	}
 
+	@Override
+	public void rewriteConfig() {
+		serverCommands().rewriteConfig();
+	}
 }

Diff for: src/main/java/org/springframework/data/redis/connection/jedis/JedisServerCommands.java

@@ -317,6 +317,11 @@ public void migrate(byte[] key, RedisNode target, int dbIndex, @Nullable Migrate
 				target.getPort(), key, dbIndex, timeoutToUse);
 	}
 
+	@Override
+	public void rewriteConfig() {
+		connection.invokeStatus().just(Jedis::configRewrite);
+	}
+
 	private boolean isPipelined() {
 		return connection.isPipelined();
 	}

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

@@ -137,6 +137,11 @@ <T, R> LettuceResult<T, R> newLettuceStatusResult(Future<T> resultHolder) {
 		return LettuceResultBuilder.<T, R> forResponse(resultHolder).buildStatusResult();
 	}
 
+	@Override
+	public void rewriteConfig() {
+		serverCommands().rewriteConfig();
+	}
+
 	private class LettuceTransactionResultConverter<T> extends TransactionResultConverter<T> {
 		public LettuceTransactionResultConverter(Queue<FutureResult<T>> txResults,
 				Converter<Exception, DataAccessException> exceptionConverter) {

Diff for: src/main/java/org/springframework/data/redis/connection/lettuce/LettuceServerCommands.java

@@ -304,6 +304,11 @@ public void migrate(byte[] key, RedisNode target, int dbIndex, @Nullable Migrate
 		connection.invoke().just(RedisKeyAsyncCommands::migrate, target.getHost(), target.getPort(), key, dbIndex, timeout);
 	}
 
+	@Override
+	public void rewriteConfig() {
+		connection.invoke().just(RedisServerAsyncCommands::configRewrite);
+	}
+
 	public RedisClusterCommands<byte[], byte[]> getConnection() {
 		return connection.getConnection();
 	}

Diff for: src/test/java/org/springframework/data/redis/connection/RedisConnectionUnitTests.java

@@ -983,6 +983,11 @@ public void migrate(byte[] key, RedisNode target, int dbIndex, MigrateOption opt
 			delegate.migrate(key, target, dbIndex, option, timeout);
 		}
 
+		@Override
+		public void rewriteConfig() {
+			delegate.rewriteConfig();
+		}
+
 		@Override
 		public Boolean set(byte[] key, byte[] value, Expiration expiration, SetOption options) {
 			return delegate.set(key, value, expiration, options);