Skip to content

Add GET parameter to SET command#1405

Merged
oranagra merged 1 commit intoredis:masterfrom
nykolaslima:add-get-parameter-to-set
Oct 2, 2020
Merged

Add GET parameter to SET command#1405
oranagra merged 1 commit intoredis:masterfrom
nykolaslima:add-get-parameter-to-set

Conversation

@nykolaslima
Copy link
Contributor

Closes #2349

Add optional GET parameter to SET command in order to set a new value to
a key and retrieve the old key value. With this change we can deprecate
GETSET command and use only the SET command with the GET parameter.

Copy link
Member

@oranagra oranagra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@itamarhaber please have a look too

@nykolaslima nykolaslima force-pushed the add-get-parameter-to-set branch from e732702 to 87bc4fe Compare September 29, 2020 21:07
Closes redis#2349

Add optional GET parameter to SET command in order to set a new value to
a key and retrieve the old key value. With this change we can deprecate
`GETSET` command and use only the SET command with the GET parameter.
@nykolaslima nykolaslima force-pushed the add-get-parameter-to-set branch from 87bc4fe to a386180 Compare September 29, 2020 21:13
@itamarhaber
Copy link
Member

LGTM

@oranagra oranagra merged commit 4a6aaa3 into redis:master Oct 2, 2020
@andy-k andy-k mentioned this pull request Jan 11, 2021
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.

3 participants