From 24e1a711b38ab597e6519caf88fd397f13675136 Mon Sep 17 00:00:00 2001 From: James Xin Date: Thu, 24 Oct 2024 12:44:21 -0700 Subject: [PATCH] address comments Signed-off-by: James Xin --- .../api/commands/servermodules/Json.java | 28 +++++++++++-------- 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/java/client/src/main/java/glide/api/commands/servermodules/Json.java b/java/client/src/main/java/glide/api/commands/servermodules/Json.java index 2d0abe823b..b525835c6c 100644 --- a/java/client/src/main/java/glide/api/commands/servermodules/Json.java +++ b/java/client/src/main/java/glide/api/commands/servermodules/Json.java @@ -191,11 +191,12 @@ public static CompletableFuture get( *
    *
  • For JSONPath (path starts with $): Returns a stringified JSON list * replies for every possible path, or a string representation of an empty array, - * if path doesn't exist. If key doesn't exist, returns null. + * if path doesn't exist. If key doesn't exist, returns null + * . *
  • For legacy path (path doesn't start with $): Returns a string * representation of the value in paths. If paths * doesn't exist, an error is raised. If key doesn't exist, returns - * null. + * null. *
*
  • If multiple paths are given: Returns a stringified JSON, in which each path is a key, * and it's corresponding value, is the value as if the path was executed in the command @@ -228,11 +229,12 @@ public static CompletableFuture get( *
      *
    • For JSONPath (path starts with $): Returns a stringified JSON list * replies for every possible path, or a string representation of an empty array, - * if path doesn't exist. If key doesn't exist, returns null. + * if path doesn't exist. If key doesn't exist, returns null + * . *
    • For legacy path (path doesn't start with $): Returns a string * representation of the value in paths. If paths * doesn't exist, an error is raised. If key doesn't exist, returns - * null. + * null. *
    *
  • If multiple paths are given: Returns a stringified JSON, in which each path is a key, * and it's corresponding value, is the value as if the path was executed in the command @@ -319,11 +321,12 @@ public static CompletableFuture get( *
      *
    • For JSONPath (path starts with $): Returns a stringified JSON list * replies for every possible path, or a string representation of an empty array, - * if path doesn't exist. If key doesn't exist, returns null. + * if path doesn't exist. If key doesn't exist, returns null + * . *
    • For legacy path (path doesn't start with $): Returns a string * representation of the value in paths. If paths * doesn't exist, an error is raised. If key doesn't exist, returns - * null. + * null. *
    *
  • If multiple paths are given: Returns a stringified JSON, in which each path is a key, * and it's corresponding value, is the value as if the path was executed in the command @@ -365,11 +368,12 @@ public static CompletableFuture get( *
      *
    • For JSONPath (path starts with $): Returns a stringified JSON list * replies for every possible path, or a string representation of an empty array, - * if path doesn't exist. If key doesn't exist, returns null. + * if path doesn't exist. If key doesn't exist, returns null + * . *
    • For legacy path (path doesn't start with $): Returns a string * representation of the value in paths. If paths * doesn't exist, an error is raised. If key doesn't exist, returns - * null. + * null. *
    *
  • If multiple paths are given: Returns a stringified JSON, in which each path is a key, * and it's corresponding value, is the value as if the path was executed in the command @@ -1217,7 +1221,7 @@ public static CompletableFuture toggle( * @param client The Valkey GLIDE client to execute the command. * @param key The key of the JSON document. * @return Returns the JSON document in its RESP form. - * If key doesn't exist, null is returned. + * If key doesn't exist, null is returned. * @example *
    {@code
          * Json.set(client, "doc", ".", "{\"a\": [1, 2, 3], \"b\": {\"b1\": 1}, \"c\": 42}");
    @@ -1245,7 +1249,7 @@ public static CompletableFuture resp(@NonNull BaseClient client, @NonNul
          * @param client The Valkey GLIDE client to execute the command.
          * @param key The key of the JSON document.
          * @return Returns the JSON document in its RESP form.
    -     *     If key doesn't exist, null is returned.
    +     *     If key doesn't exist, null is returned.
          * @example
          *     
    {@code
          * Json.set(client, "doc", ".", "{\"a\": [1, 2, 3], \"b\": {\"b1\": 1}, \"c\": 42}");
    @@ -1286,7 +1290,7 @@ public static CompletableFuture resp(
          *           paths match, the value of the first JSON value match is returned. If path
          *            doesn't exist, an error is raised.
          *     
    -     *     If key doesn't exist, null is returned.
    +     *     If key doesn't exist, null is returned.
          * @example
          *     
    {@code
          * Json.set(client, "doc", ".", "{\"a\": [1, 2, 3], \"b\": {\"a\": [1, 2], \"c\": {\"a\": 42}}}");
    @@ -1323,7 +1327,7 @@ public static CompletableFuture resp(
          *           paths match, the value of the first JSON value match is returned. If path
          *            doesn't exist, an error is raised.
          *     
    -     *     If key doesn't exist, null is returned.
    +     *     If key doesn't exist, null is returned.
          * @example
          *     
    {@code
          * Json.set(client, "doc", ".", "{\"a\": [1, 2, 3], \"b\": {\"a\": [1, 2], \"c\": {\"a\": 42}}}");