Time taken to return to application from hiredis library seems to be too long (>60sec)

[rahulKrishnaM]

Time taken to return to application from hiredis library seems to be too long (>60sec)

[bjosv]

Update:
I'm currently trying out a change for using async reconnection attempts.
Hopefully this will fix this problem, but I need to try it out a bit more.

[bjosv]

This relates with #19 and the conditions haven't changed since.
I guess you have a pretty low connect timeout already?

[rahulKrishnaM]

Ya connect-timeout is set to 2seconds currently.

[rahulKrishnaM]

So, a blocking connect() call will effectively see a max wait time of connect_timeout value, right?
And, if a poll() follows a connect(), i guess there too we will see a max of connect_timeout wait time. So, per redis server-ip there could be a 2*connect_timeout wait time in an error case.

[bjosv]

I'm not following you regarding the following poll()? The poll() in hiredis is the one that handles timeout I though.

I guess the amount of outstanding callbacks add-on time, depending on max-retry-count.
If max-retry-count is higher then fewer callbacks triggers a config-get, so that might be a remedy..

[rahulKrishnaM]

I'm not following you regarding the following poll()? The poll() in hiredis is the one that handles timeout I though.

what I meant was this piece of code, which follows after a failed connect() call.
https://github.com/redis/hiredis/blob/master/net.c#L266

[rahulKrishnaM]

I'm not following you regarding the following poll()? The poll() in hiredis is the one that handles timeout I though.

what I meant was this piece of code, which follows after a failed connect() call. https://github.com/redis/hiredis/blob/master/net.c#L266

Hope I made this clear @bjosv

[rahulKrishnaM]

Thanks for the update @bjosv . Making the reconnection attempts async would really help in improving this wait period. Shall wait for the changes to retry the scenario.

[bjosv]

A change covering this issue is now delivered, hope it fixes the problems in your setup as well.

[rahulKrishnaM]

Nice! Thanks, bjosv, shall try pulling this and see how it goes.