mirror of
https://github.com/serai-dex/serai.git
synced 2025-12-08 12:19:24 +00:00
Add note to signers on reducing disk IO
This commit is contained in:
@@ -22,6 +22,19 @@ db_channel! {
|
|||||||
SlashReport: (session: Session) -> Vec<Slash>,
|
SlashReport: (session: Session) -> Vec<Slash>,
|
||||||
SlashReportSignature: (session: Session) -> Vec<u8>,
|
SlashReportSignature: (session: Session) -> Vec<u8>,
|
||||||
|
|
||||||
|
/*
|
||||||
|
TODO: Most of these are pointless? We drop all active signing sessions on reboot. It's
|
||||||
|
accordingly not valuable to use a DB-backed channel to communicate messages for signing
|
||||||
|
sessions (Preprocess/Shares).
|
||||||
|
|
||||||
|
Transactions, Batches, Slash Reports, and Cosigns all have their own mechanisms/DB entries
|
||||||
|
and don't use the following channels. The only questions are:
|
||||||
|
|
||||||
|
1) If it's safe to drop Reattempt? Or if we need tweaks to enable that
|
||||||
|
2) If we reboot with a pending Reattempt, we'll participate on reboot. If we drop that
|
||||||
|
Reattempt, we won't. Accordingly, we have degraded performance in that edge case in
|
||||||
|
exchange for less disk IO in the majority of cases. Is that work it?
|
||||||
|
*/
|
||||||
CoordinatorToCosignerMessages: (session: Session) -> CoordinatorMessage,
|
CoordinatorToCosignerMessages: (session: Session) -> CoordinatorMessage,
|
||||||
CosignerToCoordinatorMessages: (session: Session) -> ProcessorMessage,
|
CosignerToCoordinatorMessages: (session: Session) -> ProcessorMessage,
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user