Commit d922ae65 authored by antirez's avatar antirez
Browse files

Now SORT returns an empty bulk reply if the key does not exist

parent 04c71068
...@@ -4839,7 +4839,7 @@ static void sortCommand(redisClient *c) { ...@@ -4839,7 +4839,7 @@ static void sortCommand(redisClient *c) {
/* Lookup the key to sort. It must be of the right types */ /* Lookup the key to sort. It must be of the right types */
sortval = lookupKeyRead(c->db,c->argv[1]); sortval = lookupKeyRead(c->db,c->argv[1]);
if (sortval == NULL) { if (sortval == NULL) {
addReply(c,shared.nokeyerr); addReply(c,shared.nullmultibulk);
return; return;
} }
if (sortval->type != REDIS_SET && sortval->type != REDIS_LIST && if (sortval->type != REDIS_SET && sortval->type != REDIS_LIST &&
......
...@@ -962,6 +962,10 @@ proc main {server port} { ...@@ -962,6 +962,10 @@ proc main {server port} {
$r sort mylist BY weight_* GET # $r sort mylist BY weight_* GET #
} {2 1 3} } {2 1 3}
test {SORT with constant GET} {
$r sort mylist GET foo
} {{} {} {}}
test {LREM, remove all the occurrences} { test {LREM, remove all the occurrences} {
$r flushdb $r flushdb
$r rpush mylist foo $r rpush mylist foo
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment