♻️ Streaming node state #1535
♻️ Streaming node state #1535
Conversation
| fn on_event(&mut self, event: &TxEvent); | ||
| } | ||
|
|
||
| pub struct BlockNodeStateCallback { |
There was a problem hiding this comment.
Pourquoi ne pas l'implémenter directement sur Block, orphan rule again ?
There was a problem hiding this comment.
Ça m'était juste pas venu à l'idée. En soit c'est temporaire, je vais voir si je rechange.
| Ok(()) | ||
| } | ||
|
|
||
| pub fn handle_signed_block(&mut self, signed_block: &SignedBlock) -> Result<Block> { |
There was a problem hiding this comment.
Pourquoi store un truc stateful ? On pourrait générer le block dans la fonction directement non ?
There was a problem hiding this comment.
J'ai fait comme ça parce que ça me paraissait potentiellement plus facile que de passer le truc en argument partout, mais ça se débat en vrai
| ContractTimeoutWindowUpdated(&'a TxId, &'a ContractName, &'a TimeoutWindow), | ||
| } | ||
|
|
||
| pub trait NodeStateCallback: std::any::Any { |
There was a problem hiding this comment.
NodeStateInterpreter ? (Callback ça évoque un truc asynchrone)
There was a problem hiding this comment.
Toutes façons cette struct est temporaire, mais ui
wraitii
force-pushed
the
streaming_node_state
branch
3 times, most recently
from
f47436b to
4ca5b62
Compare
wraitii
force-pushed
the
streaming_node_state
branch
from
2868363 to
82c258f
Compare
No description provided.