search

coolexp / redis

Automatically exported from code.google.com/p/redis
0 stars 0 forks source link

Response bytesize is returned as $<size> #36

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
jcarouth@jcarouth-laptop:~/workspace/Carouth_Redis/tests> telnet localhost 6379
Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
SET foo 3
bar
+OK
GET foo
$3
bar

What is the expected output? What do you see instead?
Expected
--------------
3
bar

Actual
--------------
$3
bar

What version of the product are you using? On what operating system?
0.096
Linux jcarouth-laptop 2.6.27.21-0.1-pae #1 SMP 2009-03-31 14:50:44 +0200
i686 i686 i386 GNU/Linux

Additional Information
I am not sure if this is a bug or not, but the documentation and example
PHP client assumes the $ is not present.

Original issue reported on code.google.com by jcaro...@gmail.com on 10 May 2009 at 3:24

GoogleCodeExporter commented 8 years ago 
Hello, this is the new protocol, but actually it is this way since at least 
Redis
0.09x and the documentation should reflect the new protocol. Idem for the PHP 
lib.
May you please report where the doc or the code is not uptodate?

Thanks!

Original comment by anti...@gmail.com on 10 May 2009 at 8:27

GoogleCodeExporter commented 8 years ago 
Right you are, antierz. I was looking at the PHP client library in the 
repository on
googlecode. The client on github behaves as expected.

I also do not know which documentation I was looking at and I can't seem to 
find it.
I guess just reject this ticket.

Thanks!

Original comment by jcaro...@gmail.com on 11 May 2009 at 2:09

GoogleCodeExporter commented 8 years ago 
I found where the documentation needs updating: 

http://code.google.com/p/redis/wiki/README

The Redis Tutorial section has a simple example that does not follow the 
protocol
spec. Not a big deal, but for beginners could be fairly confusing.

Original comment by jcaro...@gmail.com on 11 May 2009 at 6:35

GoogleCodeExporter commented 8 years ago 
README fixed

Original comment by anti...@gmail.com on 22 May 2009 at 4:44