Skip to content

update data model, SUP, and BSUP specs - #7628

Merged
mccanne merged 2 commits into
mainfrom
update-bsup-spec
Nov 10, 2026
Merged

mccanne merged 2 commits into
mainfrom
update-bsup-spec

Conversation

@mccanne

@mccanne mccanne commented Nov 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@philrz philrz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I re-read everything that was touched here and spotted just a couple small suggested edits near what was changed, but otherwise looks good! 👍

Comment thread book/src/formats/model.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* _set_ - an unordered collection of zero or more unique values called elements,

Comment thread book/src/super-sql/types/enum.md Outdated
symbols must be enumerated anywhere the type appears. In the binary formats
of BSUP and CSUP, the enum symbols are encoded efficiently just once.
symbols must be enumerated anywhere the type appears. In the binary
BSUP format, the enum symbols are encoded efficiently just once.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
BSUP format, the enum symbols are encoded efficiently just once.
[BSUP](../../formats/bsup.md) format, the enum symbols are encoded efficiently just once.

@mccanne
mccanne merged commit 7ca99e2 into main Nov 10, 2026
4 checks passed
@mccanne
mccanne deleted the update-bsup-spec branch November 10, 2026 00:01
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