Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Dispute coordinator overlay #3462

Merged

Conversation

Lldenaurois
Copy link
Contributor

This diff moves the dispute coordinator to the common Backend, OverlayedBackend pattern used in both the Approval Voting and Chain Selection subsystems.

In preparation of moving to the overlayed backend pattern, this commit
moves the db to return SubsystemResult values.
This commit adds the backend and overlayed backend structs to the
dispute-coordinator subsystem.
This commit finalizes the move from the previous transactional model
to the common overlay pattern in subsystem persistency. This can be
observed in the ApprovalVoting and ChainSelection subsystems.
@Lldenaurois Lldenaurois added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. I8-refactor Code needs refactoring. labels Jul 12, 2021
@@ -0,0 +1,160 @@
use polkadot_primitives::v1::{CandidateHash, SessionIndex};
Copy link
Contributor

Choose a reason for hiding this comment

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

License header & module docs, please

@rphmeier
Copy link
Contributor

Looks good! Needs another merge; last one must have missed something.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. I8-refactor Code needs refactoring.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants