validation: only call UpdatedBlockTip for active chainstate
AcceptedPublic
Actions

Authored by PiRK on Thu, Apr 3, 13:31.

Details

Reviewers

roqqit

Group Reviewers

Restricted Project

Summary

This notification isn't needed for background chainstates.

This is a partial backport of core#27596
https://github.com/bitcoin/bitcoin/pull/27596/commits/1e59acdf17309f567c370885f0cf02605e2baa58

Depends on D17888

Test Plan

tested on dev branch via the feature_assumeutxo functional test and a test on mainnet (including indexers)

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 32884
Build 65252: Build Diff	build-without-wallet · build-diff · lint-circular-dependencies · build-clang · build-clang-tidy · build-debug
Build 65251: arc lint + arc unit

Event Timeline

PiRK created this revision.Thu, Apr 3, 13:31

Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Apr 3, 13:31

PiRK requested review of this revision.Thu, Apr 3, 13:31

teamcity edited the summary of this revision. (Show Details)Thu, Apr 3, 13:31

Harbormaster completed remote builds in B32884: Diff 53355.Thu, Apr 3, 13:46

roqqit accepted this revision.Thu, Apr 3, 17:03

This revision is now accepted and ready to land.Thu, Apr 3, 17:03

roqqit added a child revision: D17890: validation: pass ChainstateRole for validationinterface calls.Thu, Apr 3, 17:05

Revision Contents
Changeset List

Path

Size

src/

validation.cpp

3 lines

validationinterface.h

2 lines

Diff 53355

View Options

src/validation.cpp