Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Propagate cause for "Cluster retry deadline exceeded" exception #4103

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ggivo
Copy link
Contributor

@ggivo ggivo commented Feb 26, 2025

Improve troubleshooting in case of JedisCluster "Cluster retry deadline exceeded" error.

See: #3613

@ggivo ggivo requested a review from Copilot February 26, 2025 21:24

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Overview

This PR improves troubleshooting by propagating the underlying exception as the cause when the cluster retry deadline is exceeded.

  • Added the lastException as a cause in JedisClusterOperationException.
  • Enhances error detail for debugging clustered Redis connection issues.

Reviewed Changes

File Description
src/main/java/redis/clients/jedis/executors/ClusterCommandExecutor.java Propagates the underlying exception cause to improve error visibility

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@ggivo ggivo assigned ggivo and unassigned tishun Feb 26, 2025
@ggivo ggivo requested a review from tishun February 26, 2025 21:46
@ggivo ggivo added this to the 6.0.0 milestone Feb 26, 2025
Copy link
Contributor

@tishun tishun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants