Textual delete command w/ expire time & noreply in binary protocol version about memcached

[GoogleCodeExporter]

The support for expire time has been removed in delete. See
http://github.com/dustin/memcached/tree/rewritten-bin/doc/protocol.txt 

Then both delete commands should return "ERROR\r\n" ?
Or maybe a more appropriate return message would be "CLIENT_ERROR <error>\r\n" 
for both?

Original comment by [email protected] on 4 Dec 2008 at 12:19

[GoogleCodeExporter]

It's unclear to me what the remaining bug is here.  If there's still a bug 
here, can
we update the summary so that we know what it is?

(acknowledging there's confusion around the disappearance of delete with 
timeout)

Original comment by [email protected] on 22 Jan 2009 at 1:21

[GoogleCodeExporter]

I think the only change under 2) should be returning "CLIENT_ERROR <error>\r\n"
instead of just "ERROR\r\n". Just because it's a more appropriate error message 
in
this case. I'm not sure about 1). We could keep it the way it is - i.e. 
accepting the
expire time in the delete command even if this means accepting a wrongly 
formatted
command. Maybe that's ok because of backward compatibility issues.

Original comment by [email protected] on 22 Jan 2009 at 8:19

[GoogleCodeExporter]

I've pushed a branch for this (finally... this is an old one).

Original comment by [email protected] on 29 Oct 2009 at 6:09

[GoogleCodeExporter]

Original comment by [email protected] on 29 Oct 2009 at 6:09

• Changed state: Started

[GoogleCodeExporter]

1) stats-detail and udp test fails
2) I liked that you wrote a better error message, but could you change it to:
"CLIENT_ERROR bad command line format. Usage: delete <key> [noreply]"

This would let the user know how to send the command without having to open the
protocol spec... ("bad command line format" could be replaced with "Syntax 
error" if
you want to shorten the error message ;-))

Original comment by [email protected] on 29 Oct 2009 at 1:01

[GoogleCodeExporter]

Original comment by [email protected] on 29 Oct 2009 at 5:14

• Changed state: Fixed