You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by "PragmaTwice (via GitHub)" <gi...@apache.org> on 2023/10/15 10:07:28 UTC
[I] JSON.ARRAPPEND [kvrocks]
PragmaTwice opened a new issue, #1806:
URL: https://github.com/apache/kvrocks/issues/1806
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add support for the JSON.ARRAPPEND command [kvrocks]
Posted by "mapleFU (via GitHub)" <gi...@apache.org>.
mapleFU commented on issue #1806:
URL: https://github.com/apache/kvrocks/issues/1806#issuecomment-1772127900
Oh I found a interesting thing in RedisJson...
```
///
/// JSON.ARRAPPEND <key> <path> <json> [json ...]
///
pub fn json_arr_append<M: Manager>(
manager: M,
ctx: &Context,
args: Vec<RedisString>,
) -> RedisResult;
```
The code of it is `JSON.ARRAPPEND <key> <path> <json>`. But the document of it says the path is optional, it's so misleading...
Our current impl is right
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add support for the JSON.ARRAPPEND command [kvrocks]
Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk commented on issue #1806:
URL: https://github.com/apache/kvrocks/issues/1806#issuecomment-1768554145
@lieck Assigned, thank you!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add support for the JSON.ARRAPPEND command [kvrocks]
Posted by "lieck (via GitHub)" <gi...@apache.org>.
lieck commented on issue #1806:
URL: https://github.com/apache/kvrocks/issues/1806#issuecomment-1768536260
I want to take this issue, thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Add support for the JSON.ARRAPPEND command [kvrocks]
Posted by "mapleFU (via GitHub)" <gi...@apache.org>.
mapleFU closed issue #1806: Add support for the JSON.ARRAPPEND command
URL: https://github.com/apache/kvrocks/issues/1806
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org