Polish JSON API documentation and formatting #3369

- Remove JavaDoc comments from RedisJsonCommandBuilder String methods to match class conventions
- Update @author entries to use full names (SeungSu Kim)
- Fix @SInCE annotation from 6.5 to 6.9 for new String overload methods
- Restore blank lines and fix spacing to maintain consistent formatting
- Align documentation style with existing codebase patterns

Author: KoKimSS

src/main/java/io/lettuce/core/AbstractRedisAsyncCommands.java

@@ -89,7 +89,7 @@
  * @author dengliming
  * @author Andrey Shlykov
  * @author Ali Takavci
- * @author KoKimSS
+ * @author SeugnSu Kim
  */
 @SuppressWarnings("unchecked")
 public abstract class AbstractRedisAsyncCommands<K, V> implements RedisAclAsyncCommands<K, V>, RedisHashAsyncCommands<K, V>,
@@ -3846,4 +3846,5 @@ private byte[] encodeScript(String script) {
         LettuceAssert.notEmpty(script, "Lua script must not be empty");
         return script.getBytes(getConnection().getOptions().getScriptCharset());
     }
+
 }

src/main/java/io/lettuce/core/AbstractRedisReactiveCommands.java

@@ -98,7 +98,7 @@
  * @author Andrey Shlykov
  * @author Ali Takavci
  * @author Tihomir Mateev
- * @author KokimSS
+ * @author SeugnSu Kim
  * @since 4.0
  */
 public abstract class AbstractRedisReactiveCommands<K, V>

src/main/java/io/lettuce/core/RedisJsonCommandBuilder.java

@@ -30,7 +30,7 @@
  * Implementation of the {@link BaseRedisCommandBuilder} handling JSON commands.
  *
  * @author Tihomir Mateev
- * @author KoKimSS
+ * @author SeugnSu Kim
  * @since 6.5
  */
 class RedisJsonCommandBuilder<K, V> extends BaseRedisCommandBuilder<K, V> {
@@ -59,15 +59,6 @@ Command<K, V, List<Long>> jsonArrappend(K key, JsonPath jsonPath, JsonValue... j
         return createCommand(JSON_ARRAPPEND, (CommandOutput) new ArrayOutput<>(codec), args);
     }
 
-    /**
-     * JSON.ARRAPPEND command with raw string values to avoid unnecessary conversions.
-     *
-     * @param key        the key
-     * @param jsonPath   the JSON path
-     * @param jsonValues the raw JSON string values
-     * @return the command
-     * @since 6.5
-     */
     Command<K, V, List<Long>> jsonArrappend(K key, JsonPath jsonPath, String... jsonValues) {
         notNullKey(key);
 
@@ -101,16 +92,6 @@ Command<K, V, List<Long>> jsonArrindex(K key, JsonPath jsonPath, JsonValue value
         return createCommand(JSON_ARRINDEX, (CommandOutput) new ArrayOutput<>(codec), args);
     }
 
-    /**
-     * JSON.ARRINDEX command with raw string value to avoid unnecessary conversions.
-     *
-     * @param key      the key
-     * @param jsonPath the JSON path
-     * @param value    the raw JSON string value
-     * @param range    the range arguments
-     * @return the command
-     * @since 6.5
-     */
     Command<K, V, List<Long>> jsonArrindex(K key, JsonPath jsonPath, String value, JsonRangeArgs range) {
         notNullKey(key);
 
@@ -142,16 +123,6 @@ Command<K, V, List<Long>> jsonArrinsert(K key, JsonPath jsonPath, int index, Jso
         return createCommand(JSON_ARRINSERT, (CommandOutput) new ArrayOutput<>(codec), args);
     }
 
-    /**
-     * JSON.ARRINSERT command with raw string values to avoid unnecessary conversions.
-     *
-     * @param key      the key
-     * @param jsonPath the JSON path
-     * @param index    the index
-     * @param values   the raw JSON string values
-     * @return the command
-     * @since 6.5
-     */
     Command<K, V, List<Long>> jsonArrinsert(K key, JsonPath jsonPath, int index, String... values) {
         notNullKey(key);
 
@@ -258,15 +229,6 @@ Command<K, V, String> jsonMerge(K key, JsonPath jsonPath, JsonValue value) {
         return createCommand(JSON_MERGE, new StatusOutput<>(codec), args);
     }
 
-    /**
-     * JSON.MERGE command with raw string value to avoid unnecessary conversions.
-     *
-     * @param key      the key
-     * @param jsonPath the JSON path
-     * @param value    the raw JSON string value
-     * @return the command
-     * @since 6.5
-     */
     Command<K, V, String> jsonMerge(K key, JsonPath jsonPath, String value) {
         notNullKey(key);
 
@@ -353,16 +315,6 @@ Command<K, V, String> jsonSet(K key, JsonPath jsonPath, JsonValue value, JsonSet
         return createCommand(JSON_SET, new StatusOutput<>(codec), args);
     }
 
-    /**
-     * JSON.SET command with raw string value to avoid unnecessary conversions.
-     *
-     * @param key      the key
-     * @param jsonPath the JSON path
-     * @param value    the raw JSON string value
-     * @param options  the optional {@link JsonSetArgs}
-     * @return the command
-     * @since 6.5
-     */
     Command<K, V, String> jsonSet(K key, JsonPath jsonPath, String value, JsonSetArgs options) {
         notNullKey(key);
 

src/main/java/io/lettuce/core/api/async/RedisJsonAsyncCommands.java

@@ -22,7 +22,7 @@
  * @param <K> Key type.
  * @param <V> Value type.
  * @author Tihomir Mateev
- * @author KoKimSS
+ * @author SeugnSu Kim
  * @see <a href="https://redis.io/docs/latest/develop/data-types/json/">Redis JSON</a>
  * @since 6.5
  * @generated by io.lettuce.apigenerator.CreateAsyncApi
@@ -57,7 +57,7 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the array inside the document.
      * @param jsonStrings one or more JSON strings to be appended.
      * @return Long the resulting size of the arrays after the new data was appended, or null if the path does not exist.
-     * @since 6.5
+     * @since 6.9
      */
     RedisFuture<List<Long>> jsonArrappend(K key, JsonPath jsonPath, String... jsonStrings);
 
@@ -94,7 +94,7 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @param jsonString the JSON string to search for.
      * @param range the {@link JsonRangeArgs} to search within.
      * @return Long the index hosting the searched element, -1 if not found or null if the specified path is not an array.
-     * @since 6.5
+     * @since 6.9
      */
     RedisFuture<List<Long>> jsonArrindex(K key, JsonPath jsonPath, String jsonString, JsonRangeArgs range);
 
@@ -119,7 +119,7 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @param index the index before which the new elements will be inserted.
      * @param jsonStrings one or more JSON strings to be inserted.
      * @return Long the resulting size of the arrays after the new data was inserted, or null if the path does not exist.
-     * @since 6.5
+     * @since 6.9
      */
     RedisFuture<List<Long>> jsonArrinsert(K key, JsonPath jsonPath, int index, String... jsonStrings);
 
@@ -298,7 +298,7 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the value to merge.
      * @param jsonString the JSON string to merge.
      * @return String "OK" if the merge was successful, error if the operation failed.
-     * @since 6.5
+     * @since 6.9
      */
     RedisFuture<String> jsonMerge(K key, JsonPath jsonPath, String jsonString);
 
@@ -422,7 +422,7 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @param jsonString the JSON string to set.
      * @param options the {@link JsonSetArgs} the options for setting the value.
      * @return String "OK" if the set was successful, null if the {@link JsonSetArgs} conditions are not met.
-     * @since 6.5
+     * @since 6.9
      */
     RedisFuture<String> jsonSet(K key, JsonPath jsonPath, String jsonString, JsonSetArgs options);
 
@@ -454,7 +454,7 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the value(s) where we want to append the value.
      * @param jsonString the JSON string to append.
      * @return Long the new length of the string, or null if the matching JSON value is not a string.
-     * @since 6.5
+     * @since 6.9
      */
     RedisFuture<List<Long>> jsonStrappend(K key, JsonPath jsonPath, String jsonString);
 
@@ -507,4 +507,5 @@ public interface RedisJsonAsyncCommands<K, V> {
      * @since 6.5
      */
     RedisFuture<List<JsonType>> jsonType(K key);
+
 }

src/main/java/io/lettuce/core/api/reactive/RedisJsonReactiveCommands.java

@@ -23,7 +23,7 @@
  * @param <K> Key type.
  * @param <V> Value type.
  * @author Tihomir Mateev
- * @author KoKimSS
+ * @author SeugnSu Kim
  * @see <a href="https://redis.io/docs/latest/develop/data-types/json/">Redis JSON</a>
  * @since 6.5
  * @generated by io.lettuce.apigenerator.CreateReactiveApi
@@ -58,7 +58,7 @@ public interface RedisJsonReactiveCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the array inside the document.
      * @param jsonStrings one or more JSON strings to be appended.
      * @return Long the resulting size of the arrays after the new data was appended, or null if the path does not exist.
-     * @since 6.5
+     * @since 6.9
      */
     Flux<Long> jsonArrappend(K key, JsonPath jsonPath, String... jsonStrings);
 
@@ -95,7 +95,7 @@ public interface RedisJsonReactiveCommands<K, V> {
      * @param jsonString the JSON string to search for.
      * @param range the {@link JsonRangeArgs} to search within.
      * @return Long the index hosting the searched element, -1 if not found or null if the specified path is not an array.
-     * @since 6.5
+     * @since 6.9
      */
     Flux<Long> jsonArrindex(K key, JsonPath jsonPath, String jsonString, JsonRangeArgs range);
 
@@ -120,7 +120,7 @@ public interface RedisJsonReactiveCommands<K, V> {
      * @param index the index before which the new elements will be inserted.
      * @param jsonStrings one or more JSON strings to be inserted.
      * @return Long the resulting size of the arrays after the new data was inserted, or null if the path does not exist.
-     * @since 6.5
+     * @since 6.9
      */
     Flux<Long> jsonArrinsert(K key, JsonPath jsonPath, int index, String... jsonStrings);
 
@@ -299,7 +299,7 @@ public interface RedisJsonReactiveCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the value to merge.
      * @param jsonString the JSON string to merge.
      * @return String "OK" if the merge was successful, error if the operation failed.
-     * @since 6.5
+     * @since 6.9
      */
     Mono<String> jsonMerge(K key, JsonPath jsonPath, String jsonString);
 
@@ -423,7 +423,7 @@ public interface RedisJsonReactiveCommands<K, V> {
      * @param jsonString the JSON string to set.
      * @param options the {@link JsonSetArgs} the options for setting the value.
      * @return String "OK" if the set was successful, null if the {@link JsonSetArgs} conditions are not met.
-     * @since 6.5
+     * @since 6.9
      */
     Mono<String> jsonSet(K key, JsonPath jsonPath, String jsonString, JsonSetArgs options);
 
@@ -455,7 +455,7 @@ public interface RedisJsonReactiveCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the value(s) where we want to append the value.
      * @param jsonString the JSON string to append.
      * @return Long the new length of the string, or null if the matching JSON value is not a string.
-     * @since 6.5
+     * @since 6.9
      */
     Flux<Long> jsonStrappend(K key, JsonPath jsonPath, String jsonString);
 

src/main/java/io/lettuce/core/api/sync/RedisJsonCommands.java

@@ -21,7 +21,7 @@
  * @param <K> Key type.
  * @param <V> Value type.
  * @author Tihomir Mateev
- * @author KoKimSS
+ * @author SeugnSu Kim
  * @see <a href="https://redis.io/docs/latest/develop/data-types/json/">Redis JSON</a>
  * @since 6.5
  * @generated by io.lettuce.apigenerator.CreateSyncApi
@@ -56,7 +56,7 @@ public interface RedisJsonCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the array inside the document.
      * @param jsonStrings one or more JSON strings to be appended.
      * @return Long the resulting size of the arrays after the new data was appended, or null if the path does not exist.
-     * @since 6.5
+     * @since 6.9
      */
     List<Long> jsonArrappend(K key, JsonPath jsonPath, String... jsonStrings);
 
@@ -93,7 +93,7 @@ public interface RedisJsonCommands<K, V> {
      * @param jsonString the JSON string to search for.
      * @param range the {@link JsonRangeArgs} to search within.
      * @return Long the index hosting the searched element, -1 if not found or null if the specified path is not an array.
-     * @since 6.5
+     * @since 6.9
      */
     List<Long> jsonArrindex(K key, JsonPath jsonPath, String jsonString, JsonRangeArgs range);
 
@@ -118,7 +118,7 @@ public interface RedisJsonCommands<K, V> {
      * @param index the index before which the new elements will be inserted.
      * @param jsonStrings one or more JSON strings to be inserted.
      * @return Long the resulting size of the arrays after the new data was inserted, or null if the path does not exist.
-     * @since 6.5
+     * @since 6.9
      */
     List<Long> jsonArrinsert(K key, JsonPath jsonPath, int index, String... jsonStrings);
 
@@ -297,7 +297,7 @@ public interface RedisJsonCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the value to merge.
      * @param jsonString the JSON string to merge.
      * @return String "OK" if the merge was successful, error if the operation failed.
-     * @since 6.5
+     * @since 6.9
      */
     String jsonMerge(K key, JsonPath jsonPath, String jsonString);
 
@@ -421,7 +421,7 @@ public interface RedisJsonCommands<K, V> {
      * @param jsonString the JSON string to set.
      * @param options the {@link JsonSetArgs} the options for setting the value.
      * @return String "OK" if the set was successful, null if the {@link JsonSetArgs} conditions are not met.
-     * @since 6.5
+     * @since 6.9
      */
     String jsonSet(K key, JsonPath jsonPath, String jsonString, JsonSetArgs options);
 
@@ -453,7 +453,7 @@ public interface RedisJsonCommands<K, V> {
      * @param jsonPath the {@link JsonPath} pointing to the value(s) where we want to append the value.
      * @param jsonString the JSON string to append.
      * @return Long the new length of the string, or null if the matching JSON value is not a string.
-     * @since 6.5
+     * @since 6.9
      */
     List<Long> jsonStrappend(K key, JsonPath jsonPath, String jsonString);
 
@@ -506,4 +506,5 @@ public interface RedisJsonCommands<K, V> {
      * @since 6.5
      */
     List<JsonType> jsonType(K key);
+
 }

src/test/java/io/lettuce/core/RedisJsonCommandBuilderUnitTests.java

@@ -33,7 +33,7 @@
  * Unit tests for {@link RedisJsonCommandBuilder}.
  *
  * @author Tihomir Mateev
- * @author KoKimSS
+ * @author SeugnSu Kim
  */
 @Tag(UNIT_TEST)
 class RedisJsonCommandBuilderUnitTests {
@@ -576,14 +576,12 @@ void shouldCorrectlyConstructJsonStrappendStringOverloadRootPath() {
 
     @Test
     void shouldVerifyStringOverloadPerformanceOptimization() {
-        // Test that String overloads bypass JsonValue creation entirely
         String jsonString = "{\"test\":\"performance\"}";
 
         Command<String, String, String> command = builder.jsonSet(MY_KEY, JsonPath.ROOT_PATH, jsonString, JsonSetArgs.Builder.defaults());
         ByteBuf buf = Unpooled.directBuffer();
         command.encode(buf);
 
-        // Verify the command is constructed correctly with direct string usage
         String expected = "*4\r\n$8\r\nJSON.SET\r\n$15\r\nbikes:inventory\r\n$1\r\n$\r\n$22\r\n{\"test\":\"performance\"}\r\n";
         assertThat(buf.toString(StandardCharsets.UTF_8)).isEqualTo(expected);
     }