Differential D14572

[avalanche] Remove some useless logs
ClosedPublic
Actions

Authored by Fabien on Oct 2 2023, 14:11.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Commits

rABC91fa3a143daf: [avalanche] Remove some useless logs

Summary

The "Requesting more avalanche addresses [...]" and "Requesting compact proofs [...]" log messages are printed periodically for each peer, which brings zero information and bloats the log. Get rid of them.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Oct 2 2023, 14:11

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 2 2023, 14:11

Fabien requested review of this revision.Oct 2 2023, 14:11

bytesofman accepted this revision.Oct 2 2023, 14:12

bytesofman added a subscriber: bytesofman.

bytesofman added inline comments.

src/net_processing.cpp
1786 ↗	(On Diff #42479)	unrelated bug fix?

This revision is now accepted and ready to land.Oct 2 2023, 14:12

Fabien added inline comments.Oct 2 2023, 14:16

src/net_processing.cpp
1786 ↗	(On Diff #42479)	Clearly not a bug fix since it's a comment, but a related comment fix - the summary correctly state that it's sent to all the peers, and the comment mistakenly states it's sent to a single one