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

DECRBYFLOAT why not? #5540

Open
topilski opened this issue Nov 8, 2018 · 6 comments
Open

DECRBYFLOAT why not? #5540

topilski opened this issue Nov 8, 2018 · 6 comments

Comments

@topilski
Copy link

topilski commented Nov 8, 2018

Hi, Redis have incrbyfloat but i didn't find decrbyfloat docs. Is this command exists?

@mgravell
Copy link

mgravell commented Nov 8, 2018 via email

@topilski
Copy link
Author

topilski commented Nov 8, 2018

Hi @mgravell , thank you for reply, may be in this case need to remove decrby?

@antirez
Copy link
Contributor

antirez commented Nov 8, 2018

@topilski yes, the idea is to deprecate it at some point... It's completely useless AFAIK.

@antirez
Copy link
Contributor

antirez commented Nov 8, 2018

(but it's there since Redis 1.0 -- back then the idea was, that given that we had INCR and DECR, when INCRBY was introduced, it was odd to lack the symmetrical. Later the community started to develop and urgency about avoiding adding new commands, so the float version comes without such symmetrical command. Unfortunately backward compatibility is great but has such side effects of not removing certain errors).

@topilski
Copy link
Author

topilski commented Nov 8, 2018

@antirez thank you for clarification, i think we can close this issue. Also please update docs for TYPE command, now Redis have new type 'stream' , thank you for the good work!

@antirez
Copy link
Contributor

antirez commented Nov 8, 2018 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

No branches or pull requests

3 participants