Skip to content

p2p: add a per-message type send and receive metric (backport #9622)#230

Merged
jmalicevic merged 39 commits intov0.37.xfrom
mergify/bp/v0.37.x/pr-9622
Jan 31, 2023
Merged

p2p: add a per-message type send and receive metric (backport #9622)#230
jmalicevic merged 39 commits intov0.37.xfrom
mergify/bp/v0.37.x/pr-9622

Conversation

@jmalicevic
Copy link
Collaborator

@jmalicevic jmalicevic commented Jan 29, 2023

This is a backport of tendermint/tendermint#9622. Original PR tendermint/tendermint#9640 .

Addresses #219 .

williambanfield and others added 30 commits October 27, 2022 19:47
* p2p: ressurrect the p2p envelope and use to calculate message metric

Co-authored-by: Callum Waters <cmwaters19@gmail.com>
(cherry picked from commit 09b8708)

# Conflicts:
#	cmd/tendermint/commands/rollback.go
#	go.mod
#	go.sum
#	p2p/peer.go
#	p2p/peer_set_test.go
@jmalicevic jmalicevic marked this pull request as ready for review January 29, 2023 21:56
@jmalicevic jmalicevic requested review from a team as code owners January 29, 2023 21:56
@jmalicevic
Copy link
Collaborator Author

I would add a changelog entry for this change, seems like a not so minor change to developers.

@thanethomson
Copy link
Contributor

I would add a changelog entry for this change, seems like a not so minor change to developers.

Agreed! I'd say the major change here is the replacement of the Send and TrySend methods in the Peer interface with SendEnvelope and TrySendEnvelope: https://github.com/cometbft/cometbft/pull/230/files#diff-56fdd197da1151147beaf6e468b618992fb58797786d85dfe2c2c6a42515d94e

@jmalicevic jmalicevic merged commit 9eaefb4 into v0.37.x Jan 31, 2023
@jmalicevic jmalicevic deleted the mergify/bp/v0.37.x/pr-9622 branch January 31, 2023 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants