From 228648c2ad70b46ba7ce13d9c8a96287ac033570 Mon Sep 17 00:00:00 2001
From: Marcin Grzejszczak <marcin@grzejszczak.pl>
Date: Mon, 25 Nov 2024 14:12:37 +0100
Subject: [PATCH 1/2] Adds missing javadocs for has key

fixes gh-3049
---
 .../org/springframework/data/redis/core/RedisOperations.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/springframework/data/redis/core/RedisOperations.java b/src/main/java/org/springframework/data/redis/core/RedisOperations.java
index 0869015a8f..6b7a6f483f 100644
--- a/src/main/java/org/springframework/data/redis/core/RedisOperations.java
+++ b/src/main/java/org/springframework/data/redis/core/RedisOperations.java
@@ -178,7 +178,7 @@ <T> T execute(RedisScript<T> script, RedisSerializer<?> argsSerializer, RedisSer
 	 * Determine if given {@code key} exists.
 	 *
 	 * @param key must not be {@literal null}.
-	 * @return
+	 * @return {@literal true} if key exists. {@literal null} when used in pipeline / transaction.
 	 * @see <a href="https://redis.io/commands/exists">Redis Documentation: EXISTS</a>
 	 */
 	@Nullable

From 83d719cdc1ce62f576b59f469ad241dc472e54c4 Mon Sep 17 00:00:00 2001
From: Marcin Grzejszczak <marcin@grzejszczak.pl>
Date: Mon, 25 Nov 2024 14:30:01 +0100
Subject: [PATCH 2/2] Added missing javadocs about what the method returns and
 about nullability

fixes gh-3049
---
 .../springframework/data/redis/core/RedisOperations.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/springframework/data/redis/core/RedisOperations.java b/src/main/java/org/springframework/data/redis/core/RedisOperations.java
index 6b7a6f483f..e3ca686a1e 100644
--- a/src/main/java/org/springframework/data/redis/core/RedisOperations.java
+++ b/src/main/java/org/springframework/data/redis/core/RedisOperations.java
@@ -47,6 +47,7 @@
  * @author Todd Merrill
  * @author Chen Li
  * @author Vedran Pavic
+ * @author Marcin Grzejszczak
  */
 public interface RedisOperations<K, V> {
 
@@ -151,7 +152,7 @@ <T> T execute(RedisScript<T> script, RedisSerializer<?> argsSerializer, RedisSer
 	 * to free resources after use.
 	 *
 	 * @param callback must not be {@literal null}.
-	 * @return
+	 * @return the {@link Object result} of the operation performed in the callback or {@literal null}.
 	 * @since 1.8
 	 */
 	@Nullable
@@ -167,7 +168,7 @@ <T> T execute(RedisScript<T> script, RedisSerializer<?> argsSerializer, RedisSer
 	 * @param sourceKey must not be {@literal null}.
 	 * @param targetKey must not be {@literal null}.
 	 * @param replace whether the key was copied. {@literal null} when used in pipeline / transaction.
-	 * @return
+	 * @return {@code true} when copied successfully or {@literal null} when used in pipeline / transaction.
 	 * @see <a href="https://redis.io/commands/copy">Redis Documentation: COPY</a>
 	 * @since 2.6
 	 */
@@ -361,7 +362,7 @@ default Boolean expireAt(K key, Instant expireAt) {
 	 * Remove the expiration from given {@code key}.
 	 *
 	 * @param key must not be {@literal null}.
-	 * @return {@literal null} when used in pipeline / transaction.
+	 * @return {@code true} when persisted successfully or {@literal null} when used in pipeline / transaction.
 	 * @see <a href="https://redis.io/commands/persist">Redis Documentation: PERSIST</a>
 	 */
 	@Nullable