search

vipshop / hiredis-vip

Support redis cluster. Maintained and used at vipshop.
BSD 3-Clause "New" or "Revised" License
322 stars 173 forks source link

Add multiple keys support on EXISTS #124

Open raghuiisc opened 4 years ago

raghuiisc commented 4 years ago

Redis EXISTS support multiple keys

As of now i see that hiredis-vip does not support the same. I get below error when i try the same.

ERROR: 4 Parse command error. Cmd type: 2, state: 10, break position: 24

From further debugging i see that in file "command.c" as following:

static int redis_arg0(struct cmd *r) { switch (r->type) { case CMD_REQ_REDIS_EXISTS: <<<<<<<

bjosv commented 3 years ago

HI! We have solved this issue in our fork. The PR that maybe could be reused is here: https://github.com/Nordix/hiredis-cluster/commit/746714d15417a37d413f63064df590046108638e