Differential D14820

[avalanche] Don't log all the avahello we send
ClosedPublic
Actions

Authored by Fabien on Nov 20 2023, 08:16.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC458ea52968ab: [avalanche] Don't log all the avahello we send

Summary

This log message is redundant with the network log, and brings no value since we send it to all our peers.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

reduce_avahello_verbosity

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25681
Build 50942: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 50941: arc lint + arc unit

Event Timeline

Fabien created this revision.Nov 20 2023, 08:16

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 20 2023, 08:16

Fabien requested review of this revision.Nov 20 2023, 08:16

PiRK accepted this revision.Nov 20 2023, 08:35

This revision is now accepted and ready to land.Nov 20 2023, 08:35

Harbormaster completed remote builds in B25681: Diff 43177.Nov 20 2023, 08:41

Closed by commit rABC458ea52968ab: [avalanche] Don't log all the avahello we send (authored by Fabien). · Explain WhyNov 20 2023, 08:46

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC458ea52968ab: [avalanche] Don't log all the avahello we send.

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

3 lines

Diff 43177

View Options

[avalanche] Don't log all the avahello we sendClosedPublicActions