fix mismatch argument and return wrong value of clusterDelNodeSlots

[leonchen83]

No description provided.

[itamarhaber]

@leonchen83 good to see you dude, looks like a good fix imo

[leonchen83]

@itamarhaber
hi
have plan to upgrade rejson from RDB_TYPE_MODULE to RDB_TYPE_MODULE_2?

[itamarhaber]

Yes, but this isn't the place to discuss it ;)

Also, rereading your fix here, looks like you'll print NULL in the error? Not sure that's good actually.

[leonchen83]

@itamarhaber
same as line 4185. and according to condition !strcasecmp(c->argv[3]->ptr,"importing") && c->argc == 5
I think it should't be NULL.

[leonchen83]

wdut, @itamarhaber ,@antirez

[antirez]

Thanks @leonchen83, the return value is actually never used, so I guess this is just a theoretical fix right (which is totally fine, we may use the return value in the future). @itamarhaber not sure about the NULL problem, we use the argument printed immediately before in clusterLookupNode(c->argv[4]->ptr).

[antirez]

Both reviewed and merged. Thanks!