Skip to content
This repository was archived by the owner on Mar 9, 2026. It is now read-only.

Adds the RESET command#1426

Merged
itamarhaber merged 1 commit into
redis:masterfrom
itamarhaber:reset-command
Nov 4, 2020
Merged

Adds the RESET command#1426
itamarhaber merged 1 commit into
redis:masterfrom
itamarhaber:reset-command

Conversation

@itamarhaber

@itamarhaber itamarhaber commented Nov 1, 2020

Copy link
Copy Markdown
Member

Ref: redis/redis#7982

Missing the 'Time complexity' which could be O(N+M+Q+P+...) though.

@itamarhaber itamarhaber requested a review from yossigo November 2, 2020 01:00

@yossigo yossigo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think time complexity for meta-data (i.e. watched keys, etc.) matters, and it is already ignored with similar commands (e.g. DISCARD).

@itamarhaber

Copy link
Copy Markdown
Member Author

@yossigo re complexity: #1202

@itamarhaber itamarhaber merged commit 08904d2 into redis:master Nov 4, 2020
@itamarhaber itamarhaber deleted the reset-command branch November 4, 2020 16:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants