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

Flatbuffers serialization for Raft Messages #44

Open
hkadayam opened this issue Nov 8, 2023 · 0 comments · Fixed by #45
Open

Flatbuffers serialization for Raft Messages #44

hkadayam opened this issue Nov 8, 2023 · 0 comments · Fixed by #45
Assignees
Labels
enhancement New feature or request

Comments

@hkadayam
Copy link
Contributor

hkadayam commented Nov 8, 2023

Currently we use Protobuf serialization for Raft messages. Given that we never mutate the serialized objects and serialize them when we are sending the message and never mutate them on receiving side, flatbuffers schema could be more useful.

Note that the data channel headers are going to be flatbuffers serialized

This issue is track an effort to see if its useful to convert to flatbuffers (should be relatively straightforward)

@szmyd szmyd added this to the MileStone4.00 milestone Nov 8, 2023
@szmyd szmyd added the enhancement New feature or request label Nov 8, 2023
@szmyd szmyd self-assigned this Nov 8, 2023
This was linked to pull requests Nov 8, 2023
@szmyd szmyd closed this as completed in #45 Nov 11, 2023
@szmyd szmyd reopened this Nov 14, 2023
@szmyd szmyd modified the milestones: MileStone4.00, MileStone4.1 Dec 21, 2023
@hkadayam hkadayam modified the milestones: MileStone4.1, Milestone4.2 Jan 31, 2024
@hkadayam hkadayam removed this from the Milestone4.2 milestone May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants