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

HINCRBY should use 64bit signed integers, not 32bit ints. #957

Merged
merged 1 commit into from
Jan 26, 2025

Conversation

prvyk
Copy link
Contributor

@prvyk prvyk commented Jan 25, 2025

Per redis docs, HINCRBY should use 64bit ints, function was mistakenly limited to 32bits.


"The range of values supported by HINCRBY is limited to 64 bit signed integers."

https://redis.io/docs/latest/commands/hincrby/

@badrishc badrishc merged commit 000f7f1 into microsoft:main Jan 26, 2025
15 checks passed
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.

2 participants