blpop ResponseError: timeout is not an integer or out of range

[shanliu]

Is it compatible with <REDIS 6.0?

cargo.toml:
redis = "~0.24";

[nihohit]

https://redis.io/commands/blpop/
Starting with Redis version 6.0.0: timeout is interpreted as a double instead of an integer.

If you're using earlier redis, you'll need to pass a value that will be parsed to an integer.

[shanliu]

It may be a good idea to obtain the Redis version when you connect, and to parse the operation differently according to different versions

https://redis.io/commands/blpop/ Starting with Redis version 6.0.0: timeout is interpreted as a double instead of an integer.

If you're using earlier redis, you'll need to pass a value that will be parsed to an integer.

I don't think it's a good idea to leave the problem to the user

[nihohit]

I'm not sure what you mean here, since the choice of type is done at compile time, but PRs are always welcome 🙂

…

On Thu, 22 Feb 2024, 13:10 shanliu, ***@***.***> wrote: It may be a good idea to obtain the Redis version when you connect, and to parse the operation differently according to different versions https://redis.io/commands/blpop/ Starting with Redis version 6.0.0: timeout is interpreted as a double instead of an integer. If you're using earlier redis, you'll need to pass a value that will be parsed to an integer. — Reply to this email directly, view it on GitHub <#1060 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABDEBBUZWMSHDR2YMKWAFJ3YU4RSVAVCNFSM6AAAAABDUVISE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJZGIZDCMJXGA> . You are receiving this because you commented.Message ID: ***@***.***>

[shanliu]

I'm not sure what you mean here, since the choice of type is done at compile time, but PRs are always welcome 🙂
…
On Thu, 22 Feb 2024, 13:10 shanliu, @.> wrote: It may be a good idea to obtain the Redis version when you connect, and to parse the operation differently according to different versions https://redis.io/commands/blpop/ Starting with Redis version 6.0.0: timeout is interpreted as a double instead of an integer. If you're using earlier redis, you'll need to pass a value that will be parsed to an integer. — Reply to this email directly, view it on GitHub <#1060 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDEBBUZWMSHDR2YMKWAFJ3YU4RSVAVCNFSM6AAAAABDUVISE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJZGIZDCMJXGA . You are receiving this because you commented.Message ID: @.>

Check in blpop.., if version <6, convert f64 to integer.

[nihohit]

as I said, PRs are welcome.