Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 39 commits into from
Jan 31, 2023

Conversation

jmalicevic
Copy link
Contributor

@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
Contributor 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.

None yet

3 participants