Skip to content

fix: prevent monitor crash when CLUSTER SLOTS times out#45

Closed
hjianbo wants to merge 1 commit into
masterfrom
fix/monitor-refresh-timeout-crash
Closed

fix: prevent monitor crash when CLUSTER SLOTS times out#45
hjianbo wants to merge 1 commit into
masterfrom
fix/monitor-refresh-timeout-crash

Conversation

@hjianbo

@hjianbo hjianbo commented Mar 30, 2026

Copy link
Copy Markdown
Member

Summary

  • catch exit from eredis:q(Connection, ["CLUSTER", "SLOTS"]) in monitor refresh path
  • convert exit to normal error tuple so gen_server does not crash
  • use safe stop wrapper for connection cleanup in this path
  • add regression eunit case for refresh timeout behavior

Verification

  • /home/moo/Worksapce/emqx-enterprise/rebar3 as test eunit -g eredis_cluster_tests:monitor_reload_slots_timeout_test_ -v

@hjianbo hjianbo closed this Mar 30, 2026
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.

1 participant