HomePhabricator

log: Remove GetAdjustedTime from IBD header progress estimation

Description

log: Remove GetAdjustedTime from IBD header progress estimation

Summary:

This is a "refactor" that shouldn't change behaviour, because the two times are most likely equal. A minimum of 5 outbound peers are needed to adjust the time. And if the time is adjusted, it will be by at most 70 minutes (DEFAULT_MAX_TIME_ADJUSTMENT). Thus, the progress estimate should differ by at most 7 blocks.

This is a backport of core#24033

Test Plan: ninja all check-all

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D12910

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Jan 11 2022, 12:07
PiRKCommitted on Dec 19 2022, 14:14
PiRKPushed on Dec 19 2022, 14:15
Reviewer
Restricted Project
Differential Revision
D12910: log: Remove GetAdjustedTime from IBD header progress estimation
Parents
rABC7c8d98584d33: Add missing thread safety lock assertions in validation.{h|cpp}
Branches
Unknown
Tags
Unknown