Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for UNLINK RENAME MULTI EXEC DISCARD and SCAN commands #50

Open
ErikOne opened this issue Jul 14, 2021 · 1 comment
Open

Support for UNLINK RENAME MULTI EXEC DISCARD and SCAN commands #50

ErikOne opened this issue Jul 14, 2021 · 1 comment

Comments

@ErikOne
Copy link

ErikOne commented Jul 14, 2021

Hi

would it be possible to add support for the commands I've listed in the title ?

Thanks

@ErikOne ErikOne changed the title Support for UNLINK and RENAME Support for UNLINK RENAME MULTI EXEC DISCARD and SCAN commands Jul 14, 2021
@zuiderkwast
Copy link
Collaborator

Hi Erik. It is already possible to do these things using the "Low-level API for sending commands to specific node".

cluster_node *redisClusterGetNodeByKey(redisClusterContext *cc, char *key);
void *redisClusterCommandToNode(redisClusterContext *cc, cluster_node *node,
                                const char *format, ...);

It's not enough documented though. We would appreciate an example of for example SCAN in the README.

For UNLINK and RENAME, I think it's strait-forward to add support in the normal APIs. A PR is welcome.

There is a draft PR for MULTI-EXEC-DISCARD which stores the slot for a transaction. I think WATCH-UNWATCH should be handled in the same way. If you have suggestions for how to do this, let's discuss it in #5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants