Skip to content

Request timeout (%s) must be larger than session timeout (%s) [1.4.5] #1749

Closed as not planned
@isamaru

Description

@isamaru

In 1.4.5, session_timeout acts as a lower bound for request_timeout.

This seems excessively strict. If we use a short timeout like 2s, the heartbeat thread could simply retry, there seems to be some logic already in for that.

In the current implementation, I found these blockers:

  • check of arguments
  • timeout of heartbeat request makes the coordinator dead. We could remove this for heartbeat thread.
  • remove timed-out in-flight-requests from appropriate collections on timeout
  • allow receiving responses out of order (Correlation IDs do not match [1.4.5] #1748)

If we clear these, I think we should allow short request_timeouts.

My goal is to allow better recovery with unstable connections (which do occassionally occur in our environment)

@dpkp what do you think?

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions