Skip to content

Switch to indexed serialization#33512

Merged
antiguru merged 2 commits intoMaterializeInc:mainfrom
antiguru:columnar_indexed
Sep 4, 2025
Merged

Switch to indexed serialization#33512
antiguru merged 2 commits intoMaterializeInc:mainfrom
antiguru:columnar_indexed

Conversation

@antiguru
Copy link
Copy Markdown
Member

@antiguru antiguru commented Sep 4, 2025

Switch column from sequence to indexed serialization. The difference is that sequence stores data as a sequence of (length, data) and indexed puts all lengths first, followed by all data. We expect this to be more efficient if we do not read all data, or repeatedly borrow its serialized representation.

For details, see frankmcsherry/columnar#30

@antiguru antiguru requested a review from a team as a code owner September 4, 2025 09:05
@antiguru antiguru requested a review from petrosagg September 4, 2025 09:10
@antiguru antiguru enabled auto-merge (squash) September 4, 2025 13:57
Switch column from sequence to indexed serialization. The difference is
that sequence stores data as a sequence of (length, data) and indexed puts
all lengths first, followed by all data. We expect this to be more
efficient if we do not read all data, or repeatedly borrow its serialized
representation.

For details, see frankmcsherry/columnar#30

Signed-off-by: Moritz Hoffmann <mh@materialize.com>
Signed-off-by: Moritz Hoffmann <mh@materialize.com>
@antiguru antiguru merged commit 4c7b52e into MaterializeInc:main Sep 4, 2025
129 checks passed
@antiguru antiguru deleted the columnar_indexed branch September 4, 2025 14:30
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.

2 participants