Skip to content

core: protect ro nodes on disconnect - #252

Merged
Col-Waltz merged 1 commit into
masterfrom
Col-Waltz/protect-disconnect-on-ro
Jun 26, 2026
Merged

core: protect ro nodes on disconnect#252
Col-Waltz merged 1 commit into
masterfrom
Col-Waltz/protect-disconnect-on-ro

Conversation

@Col-Waltz

Copy link
Copy Markdown
Collaborator

Adds protection for on_disconnect trigger. After leader change it may return ERR_READONLY while deleting consumers. Now this trigger performs only local operations on ro node.

@oleg-jukovecoleg-jukovec left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please, add a changelog entry + link to the PR: #248

Comment threadqueue/abstract.lua
Comment threadqueue/abstract.lua
@Col-Waltz
Col-Waltzforce-pushed the Col-Waltz/protect-disconnect-on-ro branch from 89e9d3e to 7195d22CompareJune 11, 2026 10:54

@oleg-jukovecoleg-jukovec left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please, describe the change for a user in the CHANGELOG.md.

Comment threadt/240-ro-on-disconnect.t Outdated
Comment threadt/240-ro-on-disconnect.t Outdated
Comment threadt/240-ro-on-disconnect.t
@Col-Waltz
Col-Waltzforce-pushed the Col-Waltz/protect-disconnect-on-ro branch from 7195d22 to da46d48CompareJune 18, 2026 09:28
Comment threadCHANGELOG.md Outdated
@Col-Waltz
Col-Waltzforce-pushed the Col-Waltz/protect-disconnect-on-ro branch from da46d48 to 3b53938CompareJune 18, 2026 14:06
Comment threadqueue/abstract.lua Outdated
Comment threadqueue/abstract.lua
Adds protection for on_disconnect trigger. After leader change
it may return ERR_READONLY while deleting consumers. Now this
trigger performs only local operations on ro node.
@Col-Waltz
Col-Waltzforce-pushed the Col-Waltz/protect-disconnect-on-ro branch from 3b53938 to 26590c5CompareJune 22, 2026 22:38
@Col-Waltz
Col-Waltz requested a review from palage4aJune 22, 2026 22:42

@palage4apalage4a left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@Col-Waltz
Col-Waltz merged commit e5383bd into masterJun 26, 2026
23 checks passed
@Col-Waltz
Col-Waltz deleted the Col-Waltz/protect-disconnect-on-ro branch June 26, 2026 10:32
@Col-WaltzCol-Waltz mentioned this pull request Jul 3, 2026
Sign up for freeto 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

@Col-Waltz@oleg-jukovec@palage4a