djveremix / redis

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

redis-cli in current edge build does not support ZINTER #238

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Compile and install Redis from GitHub as of today.
2. Create sorted set with ZADD
3. Try and do ZINTER in redis-cli.

What is the expected output? What do you see instead?

Expected is for ZINTER success/failure response.
What is seen instead: "Unknown command 'zinter'"

What version of the product are you using? On what operating system?

GitHub build as of May 10, 2010

Please provide any additional information below.

From digging around, it looks like redis-cli supports ZMERGE, a command that's 
not in redis.c.. 
so this may be merely some sort of "in the middle of arranging it" snafu :-) 
ZINTER does work OK 
from other interfaces, just not redis-cli.

Original issue reported on code.google.com by pcoo...@gmail.com on 10 May 2010 at 6:16

GoogleCodeExporter commented 9 years ago
Two things:
1) redis-cli was changed to not hold a static command table, but accept 
anything as command/arguments. Therefore, it will no longer reject commands.
2) ZINTER/ZUNION have been renamed to ZINTERSTORE/ZUNIONSTORE to correctly 
reflect what they do.

I guess this issue is deprecated now, so I'm closing it :-)

Original comment by pcnoordh...@gmail.com on 26 Jun 2010 at 9:36