gaspersic/solana
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Check if poh recorder has over stepped the leader slot

Browse Source
This commit is contained in:
Pankaj Garg
2019-03-19 14:49:36 -07:00
committed by Grimes
parent ad7e727938
commit 007e17c290
1 changed files with 23 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
core/src/poh_recorder.rs
View File

@ -92,9 +92,10 @@ impl PohRecorder {
let leader_ideal_start_tick = let leader_ideal_start_tick =
target_tick.saturating_sub(self.max_last_leader_grace_ticks); target_tick.saturating_sub(self.max_last_leader_grace_ticks);
if self.tick_height() >= target_tick if self.tick_height().saturating_sub(target_tick) < self.max_last_leader_grace_ticks
|| self.max_last_leader_grace_ticks && (self.tick_height() >= target_tick
>= target_tick.saturating_sub(self.start_tick) || self.max_last_leader_grace_ticks
>= target_tick.saturating_sub(self.start_tick))
{ {
return ( return (
true, true,
@ -763,5 +764,24 @@ mod tests {
// without sending more ticks, we should be leader now // without sending more ticks, we should be leader now
assert_eq!(poh_recorder.reached_leader_tick().0, true); assert_eq!(poh_recorder.reached_leader_tick().0, true);
assert_eq!(poh_recorder.reached_leader_tick().1, 1); assert_eq!(poh_recorder.reached_leader_tick().1, 1);
// Let's test that if a node overshoots the ticks for its target
// leader slot, reached_leader_tick() will return false
// Set the leader slot 1 slot down
poh_recorder.reset(
poh_recorder.tick_height(),
bank.last_blockhash(),
4,
Some(5),
bank.ticks_per_slot(),
);
// Send remaining ticks for the slot (remember we sent extra ticks in the previous part of the test)
for _ in 0..4 * bank.ticks_per_slot() {
poh_recorder.tick();
}
// We are not the leader, as expected
assert_eq!(poh_recorder.reached_leader_tick().0, false);
} }
} }