Skip to content

TLV: better impl and a separate crate#7694

Draft
alexpyattaev wants to merge 7 commits intoanza-xyz:masterfrom
alexpyattaev:tlv
Draft

TLV: better impl and a separate crate#7694
alexpyattaev wants to merge 7 commits intoanza-xyz:masterfrom
alexpyattaev:tlv

Conversation

@alexpyattaev
Copy link
Copy Markdown

@alexpyattaev alexpyattaev commented Aug 23, 2025

Problem

  • Alpenglow team wants to use TLV to encode alpenglow vote messages
  • Existing TLV implementation in Gossip is not ideal and has some performance issues due to double handling of same bytes by bincode

Summary of Changes

  • Isolate TLV into own crate
  • Make parsers use Bytes where possible to minimize bugs

Still ToDo:

  • hook the new impl into gossip

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants