Skip to content

Comments

perf(p2p): avoid deserialization for locally published headers#251

Merged
Wondertan merged 4 commits intomainfrom
avoid-deserialization
Mar 31, 2025
Merged

perf(p2p): avoid deserialization for locally published headers#251
Wondertan merged 4 commits intomainfrom
avoid-deserialization

Conversation

@Wondertan
Copy link
Member

@Wondertan Wondertan force-pushed the avoid-deserialization branch from 92cf688 to 4e3bf0c Compare March 26, 2025 13:45
@Wondertan Wondertan marked this pull request as ready for review March 26, 2025 13:45
@Wondertan Wondertan enabled auto-merge (squash) March 26, 2025 13:46
cristaloleg
cristaloleg previously approved these changes Mar 26, 2025
Copy link
Member

@renaynay renaynay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one test would be nice 😅

@Wondertan
Copy link
Member Author

So the assumption was that the existing test covers it, but it actually does publishing without calling Broadcast.
Now it calls Broadcast covering the added new lines

cristaloleg
cristaloleg previously approved these changes Mar 29, 2025
Copy link
Contributor

@cristaloleg cristaloleg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM

@Wondertan Wondertan merged commit 12cc054 into main Mar 31, 2025
4 checks passed
@Wondertan Wondertan deleted the avoid-deserialization branch March 31, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants