Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

slot_has_updates should not check if slot is full (and connected) #27786

Closed
wants to merge 2 commits into from
Closed

slot_has_updates should not check if slot is full (and connected) #27786

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
331fdc5
slot_has_updates should not check if slot is full (and connected)
offerm Sep 14, 2022
0d7ca1c
fix fmt
offerm Sep 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions ledger/src/blockstore.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3822,14 +3822,13 @@ fn slot_has_updates(slot_meta: &SlotMeta, slot_meta_backup: &Option<SlotMeta>) -
// from block 0, which is true iff:
// 1) The block with index prev_block_index is itself part of the trunk of consecutive blocks
// starting from block 0,
slot_meta.is_connected &&
// AND either:
// 1) The slot didn't exist in the database before, and now we have a consecutive
// block for that slot
((slot_meta_backup.is_none() && slot_meta.consumed != 0) ||
(slot_meta_backup.is_none() && slot_meta.consumed != 0) ||
// OR
// 2) The slot did exist, but now we have a new consecutive block for that slot
(slot_meta_backup.is_some() && slot_meta_backup.as_ref().unwrap().consumed != slot_meta.consumed))
(slot_meta_backup.is_some() && slot_meta_backup.as_ref().unwrap().consumed != slot_meta.consumed)
}

// Creates a new ledger with slot 0 full of ticks (and only ticks).
Expand Down