You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/09/02 22:44:56 UTC

[GitHub] [incubator-kvrocks] dirkpetersen opened a new issue, #807: redis-cli --pipe syntax error

dirkpetersen opened a new issue, #807:
URL: https://github.com/apache/incubator-kvrocks/issues/807

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.
   
   
   ### Version
   
   kvrocks unstable 40989fc26dc4229bd9f797c14eaa4df07574f47e
   ubuntu 22.04
   
   
   
   ### Minimal reproduce step
   
   ```
   root@localhost:~# cat /tmp/text1.txt
   set zzzap zzzzzzzzzzzzzzzzznologin
   set vv zzzzzzzzzzzzzzzzpsbinnologin
   ```
   
   ```
   root@localhost:~# cat /tmp/text1.txt | redis-cli --pipe --verbose
   ```
   
   
   ### What did you expect to see?
   
   This is with redis or keydb 
   
   ```
   root@localhost:~# cat /tmp/text1.txt | redis-cli -h xx.79.179.186 --pipe --verbose
   All data transferred. Waiting for the last reply...
   Last reply received from server.
   errors: 0, replies: 2
   ```
   
   ### What did you see instead?
   
   root@localhost:~# cat /tmp/text1.txt | redis-cli --pipe --verbose
   All data transferred. Waiting for the last reply...
   ERR syntax error
   Last reply received from server.
   errors: 1, replies: 1
   
   
   ### Anything Else?
   
   f
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
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

[GitHub] [incubator-kvrocks] git-hulk commented on issue #807: redis-cli --pipe syntax error

Posted by GitBox <gi...@apache.org>.
git-hulk commented on issue #807:
URL: https://github.com/apache/incubator-kvrocks/issues/807#issuecomment-1236014648

   Hello @dirkpetersen, many thanks for your feedback.
   
   The root cause is that Kvrocks's inline protocol didn't support the `\r\n` separator. Will align this with Redis behavior and it only works like below:
   
   ```
   ❯ echo "set a 1\r\nset b 2\r\n" > pipes.txt
   ❯ cat pipes.txt
   set a 1
   set b 2
   
   ❯ cat pipes.txt | redis-cli -p 6666 --pipe --verbose
   All data transferred. Waiting for the last reply...
   Last reply received from server.
   errors: 0, replies: 2
   ```
   
   BTW, PR is always welcome!


-- 
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

[GitHub] [incubator-kvrocks] tisonkun closed issue #807: redis-cli --pipe syntax error

Posted by GitBox <gi...@apache.org>.
tisonkun closed issue #807: redis-cli --pipe syntax error 
URL: https://github.com/apache/incubator-kvrocks/issues/807


-- 
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