Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

fix: use operation_timeout as socket connect timeout #73

Merged
merged 1 commit into from
Nov 19, 2019
Merged

fix: use operation_timeout as socket connect timeout #73

merged 1 commit into from
Nov 19, 2019

Conversation

levy5307
Copy link
Contributor

@levy5307 levy5307 commented Nov 19, 2019

use operation_timeout as socket connect timeout, instead of Cluster.SOCK_TIMEOUT.

This fixes the problem that the client may request a disconnected replica-server when the session was unable to establish in SOCKET_TIMEOUT (aka 1 second), and finally gets ERR_SESSION_RESET error, even though its operationTimeout is 10 seconds.

This is user-unfriendly and counterintuitive because the user actually allows 10 seconds to wait, but the request didn't utilize the sufficient time budget.

@neverchanje neverchanje changed the title refactor: use operation_timeout as socket connect timeout fix: use operation_timeout as socket connect timeout Nov 19, 2019
@neverchanje neverchanje merged commit 7021c9b into XiaoMi:thrift-0.11.0-inlined Nov 19, 2019
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