Skip to content

Conversation

nkaradzhov
Copy link
Collaborator

@nkaradzhov nkaradzhov commented Jul 2, 2025

Add command timeout in the command options. It can be set via the global options when creating a client or per command when using the generic sendCommand function

Timeout only works while the command is still pending to be sent. Once a command is sent over the wire, the timeout is deleted and is never going to trigger. This decision could be changed if we deem its not good enough


Checklist

  • Does npm test pass with this change (including linting)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)? - TODO

@nkaradzhov nkaradzhov mentioned this pull request Jul 7, 2025
3 tasks
Co-authored-by: Florian Schunk <149071178+florian-schunk@users.noreply.github.com>
@nkaradzhov nkaradzhov force-pushed the add-command-timeout branch from 1ee604b to 25ce9b0 Compare July 7, 2025 08:31
@nkaradzhov nkaradzhov merged commit 65a12d5 into redis:master Jul 7, 2025
14 checks passed
@nkaradzhov nkaradzhov deleted the add-command-timeout branch July 7, 2025 08:37
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.

3 participants