octo
/
Zebra
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

state: merge SledState::{queue, process_queue}. 

We never want to call one without the other, so just do them together.
This commit is contained in:
Henry de Valence 2020-09-10 14:42:20 -07:00
parent 93586867bc
commit de2400031c
2 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
zebra-state/src/service.rs
View File

@ -56,7 +56,6 @@ impl Service<Request> for StateService {
let (rsp_tx, rsp_rx) = oneshot::channel();
self.sled.queue(QueuedBlock { block, rsp_tx });
self.sled.process_queue();
async move {
rsp_rx

11
zebra-state/src/sled_state.rs
View File

@ -60,19 +60,12 @@ impl SledState {
/// Queue a finalized block to be committed to the state.
///
/// After queueing a finalized block, call [`process_queue`] to check whether
/// the newly queued block (and any of its descendants) can be committed to
/// the state.
/// After queueing a finalized block, this method checks whether the newly
/// queued block (and any of its descendants) can be committed to the state.
pub fn queue(&mut self, queued_block: QueuedBlock) {
let prev_hash = queued_block.block.header.previous_block_hash;
self.queued_by_prev_hash.insert(prev_hash, queued_block);
}
/// Process the queue of finalized blocks, committing any that can be committed in-order.
///
/// This should be called after [`queue`], to check whether the newly queued block
/// (and any of its descendants) can be committed to the state.
pub fn process_queue(&mut self) {
// Cloning means the closure doesn't hold a borrow of &self,
// conflicting with mutable access in the loop below.
let hash_by_height = self.hash_by_height.clone();