Skip to content

Simple support for DUMP and RESTORE #405

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

alyssaruth
Copy link

Closes #401

Simple implementation for DUMP and RESTORE which just supports string keys:

  • Attempting to dump a different key type will produce WRONGTYPE Operation against a key holding the wrong kind of value
  • For RESTORE I've implemented the REPLACE and ABSTTL optional flags, but not IDLETIME or FREQ (wasn't immediately obvious to me what these were 🤷‍♀️ )

I have also:

  • Renamed the existing Dump to DebugDump to avoid confusion
  • Extracted constants for the various key types
  • Collected common validation into validCMD and plugged it in for all the existing generic commands. Happy to do this for the remainder as well if the approach looks good to you 😄

@alicebob
Copy link
Owner

alicebob commented May 19, 2025 via email

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.

Simple support for Dump / Restore
2 participants