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.

Sign up for GitHub

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

Jump to bottom

Implement a set of commands #190

Merged
merged 4 commits into from
Aug 22, 2022
Merged

Implement a set of commands #190

merged 4 commits into from
Aug 22, 2022

Conversation

danielealbano
Copy link
Owner

@danielealbano danielealbano commented Aug 22, 2022 •
edited
Loading

This PR implements a set of PRs or the issue #126 .

The following commands have been implemented in the following PR

Command Notes
DBSIZE
EXISTS
FLUSHDB Missing ASYNC
GETSET
LCS Missing IDX, MINMATCHLEN and WITHMATCHLEN
MSETNX Currently buggy, might hang the server
PTTL
TTL

In addition:

  • the DEL command has been refactored to take advantage of the implemented shortcuts
  • the HELLO command now reports an error if the AUTH or SETNAME parameters are used as they are currently unsupported

@danielealbano danielealbano added the enhancement New feature or request label Aug 22, 2022
@danielealbano danielealbano added this to the v0.2 milestone Aug 22, 2022
@danielealbano danielealbano self-assigned this Aug 22, 2022
@danielealbano danielealbano merged commit c2e17dc into main Aug 22, 2022
@danielealbano danielealbano deleted the implement-a-set-of-commands branch August 22, 2022 21:59
@danielealbano danielealbano linked an issue Aug 22, 2022 that may be closed by this pull request
Implement redis commands APPEND, properly implement the SET command, implement SET command variations (e.g. SETNX and so on) #126
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@danielealbano danielealbano

Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.2
1 participant
@danielealbano