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

Strings #8

Closed
kappsegla opened this issue Feb 2, 2023 · 0 comments · Fixed by #104
Closed

Strings #8

kappsegla opened this issue Feb 2, 2023 · 0 comments · Fixed by #104
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@kappsegla
Copy link
Contributor

kappsegla commented Feb 2, 2023

Add support for set and get for keys with string values.

SET

GET

Keys are always strings (binary safe)

Information is sent using RESP protocol.
Example for data traffic:
Request from client to server “SET foo 100”
*3\r\n$3\r\nSET\r\n$3\r\nfoo\r\n$3\r\n100\r\n

Can also be written in the inline command format:
SET foo 100\r\n

Server should then respond with:
$2\r\nOK\r\n
or
+OK\r\n

Lets ignore optional options for now when implementing this issue? Can maybe be added later as separate options?

GET returns the value of the key. If the key doesn't exist the special value nil is returned.
https://redis.io/docs/reference/protocol-spec/#resp-bulk-strings
$-1\r\n

@kappsegla kappsegla added the enhancement New feature or request label Feb 2, 2023
@kappsegla kappsegla pinned this issue Feb 2, 2023
@kappsegla kappsegla unpinned this issue Feb 3, 2023
@wassimtajeddin wassimtajeddin reopened this Feb 3, 2023
@wassimtajeddin wassimtajeddin self-assigned this Feb 3, 2023
@RobertMili RobertMili assigned RobertMili and unassigned RobertMili Feb 3, 2023
RobertMili added a commit that referenced this issue Feb 6, 2023
@RobertMili RobertMili self-assigned this Feb 6, 2023
RobertMili added a commit that referenced this issue Feb 8, 2023
without unnecessary files
@RobertMili RobertMili mentioned this issue Feb 8, 2023
@RobertMili RobertMili linked a pull request Feb 8, 2023 that will close this issue
@wassimtajeddin wassimtajeddin linked a pull request Feb 8, 2023 that will close this issue
@kappsegla kappsegla added this to the 2 milestone Feb 12, 2023
@RobertMili RobertMili linked a pull request Feb 14, 2023 that will close this issue
kappsegla added a commit that referenced this issue Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants