Skip to content

Add basic support for config set and get#98

Closed
fluffy88 wants to merge 1 commit into
fppt:masterfrom
fluffy88:feature/config-basic-get-set
Closed

Add basic support for config set and get#98
fluffy88 wants to merge 1 commit into
fppt:masterfrom
fluffy88:feature/config-basic-get-set

Conversation

@fluffy88
Copy link
Copy Markdown

Adding support for CONFIG GET x and CONFIG SET x y where x is the full configuration parameter name.
This implementation lacks support for the glob-style patterns which Redis allows.

@inponomarev
Copy link
Copy Markdown
Collaborator

Superseded by #772, which adds thin CONFIG GET/CONFIG SET support (and more). The approach here — a separate key→value namespace that round-trips — is the same idea; #772 additionally fits the current @RedisCommand architecture, wires up the behavioural parameters (lua-time-limit, proto-max-bulk-len), and adds mock-vs-real comparison tests. Closing in its favour. Thanks for the original contribution!

@inponomarev inponomarev closed this Jun 7, 2026
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

Successfully merging this pull request may close these issues.

2 participants