Skip to content

bft: enqueue a decision when its view is aborted - #742

Open
pfi79 wants to merge 1 commit into
hyperledger:mainfrom
pfi79:735-4a-decide-enqueue
Open

pfi79 wants to merge 1 commit into
hyperledger:mainfrom
pfi79:735-4a-decide-enqueue

Conversation

@pfi79

@pfi79 pfi79 commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Decide blocked when the decision channel was full and the view was being aborted: the controller loop could be inside abortView waiting for the view to stop, and the view could be waiting to enqueue its decision. The abort and the decision waited for each other.

Hand the decision to the controller in the background when the view is aborting. The decision is not dropped: MutuallyExclusiveDeliver filters a duplicate that the view changer has already delivered.

#736

Decide blocked when the decision channel was full and the view was being
aborted: the controller loop could be inside abortView waiting for the view
to stop, and the view could be waiting to enqueue its decision. The abort
and the decision waited for each other.

Hand the decision to the controller in the background when the view is
aborting. The decision is not dropped: MutuallyExclusiveDeliver filters a
duplicate that the view changer has already delivered.

Signed-off-by: Fedor Partanskiy <fredprtnsk@gmail.com>
@pfi79
pfi79 requested a review from HagarMeir October 29, 2026 15:09
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