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

feat: Unify FlatLayout with other layouts and add metadata to their flatbuffer representation #1077

Merged
merged 11 commits into from
Oct 19, 2024
26 changes: 10 additions & 16 deletions vortex-flatbuffers/flatbuffers/vortex-serde/footer.fbs
Original file line number Diff line number Diff line change
@@ -1,24 +1,18 @@
table FlatLayout {
begin: uint64;
lwwmanning marked this conversation as resolved.
Show resolved Hide resolved
end: uint64;
}

table NestedLayout {
table Layout {
children: [Layout];
encoding: uint16;
}

union LayoutVariant {
FlatLayout,
NestedLayout,
}

table Layout {
layout: LayoutVariant;
metadata: [ubyte];
}

table Footer {
layout: Layout;
row_count: uint64;
}

table Postscript {
schema_offset: uint64;
footer_offset: uint64;
}

robert3005 marked this conversation as resolved.
Show resolved Hide resolved
root_type Footer;
root_type Postscript;
root_type Footer;
Loading
Loading