Skip to content

make it possible to route messages based on priority#87

Merged
sandreim merged 2 commits intomasterfrom
alexggh/router_priority_messages
Jun 13, 2025
Merged

make it possible to route messages based on priority#87
sandreim merged 2 commits intomasterfrom
alexggh/router_priority_messages

Conversation

@alexggh
Copy link
Copy Markdown
Contributor

@alexggh alexggh commented Jun 12, 2025

Extend round_message to be able to route messages based on the priority of the message, this will be used in the overseer to send messages with high priority for paritytech/polkadot-sdk#8830.

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Copy link
Copy Markdown
Collaborator

@sandreim sandreim left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
@alexggh alexggh requested a review from drahnr June 12, 2025 16:36
Copy link
Copy Markdown
Contributor

@drahnr drahnr left a comment

Choose a reason for hiding this comment

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

LGTM

@sandreim sandreim merged commit cf91f49 into master Jun 13, 2025
7 checks passed
github-merge-queue bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Jun 17, 2025
Extend overseer to send priority messages, the new functionality is used
for sending messages on the grandpa call path when we call
dispute-coordinator and approval-voting in
finality_target_with_longest_chain to make sure we don't block
unnecessarily.

Depends on: paritytech/orchestra#87.

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Javier Viola <javier@parity.io>
Co-authored-by: Javier Viola <363911+pepoviola@users.noreply.github.com>
paritytech-release-backport-bot bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Jun 17, 2025
Extend overseer to send priority messages, the new functionality is used
for sending messages on the grandpa call path when we call
dispute-coordinator and approval-voting in
finality_target_with_longest_chain to make sure we don't block
unnecessarily.

Depends on: paritytech/orchestra#87.

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Javier Viola <javier@parity.io>
Co-authored-by: Javier Viola <363911+pepoviola@users.noreply.github.com>
(cherry picked from commit 6b5a128)
paritytech-release-backport-bot bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Jun 17, 2025
Extend overseer to send priority messages, the new functionality is used
for sending messages on the grandpa call path when we call
dispute-coordinator and approval-voting in
finality_target_with_longest_chain to make sure we don't block
unnecessarily.

Depends on: paritytech/orchestra#87.

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Javier Viola <javier@parity.io>
Co-authored-by: Javier Viola <363911+pepoviola@users.noreply.github.com>
(cherry picked from commit 6b5a128)
alvicsam pushed a commit to paritytech/polkadot-sdk that referenced this pull request Oct 17, 2025
Extend overseer to send priority messages, the new functionality is used
for sending messages on the grandpa call path when we call
dispute-coordinator and approval-voting in
finality_target_with_longest_chain to make sure we don't block
unnecessarily.

Depends on: paritytech/orchestra#87.

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Javier Viola <javier@parity.io>
Co-authored-by: Javier Viola <363911+pepoviola@users.noreply.github.com>
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.

7 participants