-
Notifications
You must be signed in to change notification settings - Fork 579
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
Serialization for sestar::chunked_fifo
#10408
Serialization for sestar::chunked_fifo
#10408
Conversation
Signed-off-by: Michal Maslanka <michal@redpanda.com>
Signed-off-by: Michal Maslanka <michal@redpanda.com>
Signed-off-by: Michal Maslanka <michal@redpanda.com>
Added test verifying if all `std::vector`, `ss::chunked_fifo` and `fragmented_vector` Signed-off-by: Michal Maslanka <michal@redpanda.com>
Signed-off-by: Michal Maslanka <michal@redpanda.com>
9961cff
to
e7898a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
sestar::chunked_fifo
sestar::chunked_fifo
Just to confirm my understanding, this really just serializes the values (plus a size header) out and not the struct itself? |
Correct! The nice part about this sort of format is that it doesn't tie you into a specific struct on the deserialization end, which gives compatibility if we want to change data structures in places later on. Most of the collections have this same format. |
Added serialization for
ss::chunked_fifo
foradl
andserde
Backports Required
Release Notes