feat!: normalize empty lists and direct Read output - #195
Draft
wackywendell wants to merge 3 commits into
Draft
Conversation
wackywendell
force-pushed
the
wendell/emit-passthrough
branch
from
August 5, 2026 17:47
2e3e615 to
0f8f140
Compare
wackywendell
force-pushed
the
wendell/emit-read-compact
branch
from
August 5, 2026 17:47
4d2151e to
386caef
Compare
wackywendell
force-pushed
the
wendell/emit-passthrough
branch
from
August 5, 2026 18:21
0f8f140 to
5e14c7c
Compare
wackywendell
force-pushed
the
wendell/emit-read-compact
branch
from
August 5, 2026 18:21
386caef to
012d39c
Compare
wackywendell
force-pushed
the
wendell/emit-read-compact
branch
from
August 5, 2026 18:59
012d39c to
e4dca7b
Compare
wackywendell
force-pushed
the
wendell/emit-passthrough
branch
from
August 5, 2026 18:59
5e14c7c to
b146a39
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR makes empty lists explicit across the text format and uses compact
=>as the canonical spelling for a named Read withDirectoutput.For example, both of these inputs describe an empty named Read schema:
Formatting writes the first form. A Read with an explicit
Emitremains explicit:Part of #34.
Changes
_for empty extension and addendum arguments, extension outputcolumns, named and virtual Read schemas, and Aggregate output.
Read[my.table +> ],Read:Virtual[() => ], andAggregate[_ => ].Directoutput as=> schema, including verboseformatting and protobuf input that explicitly contains
Direct.Read[table +> schema]asDirectinput, but format itwith
=>.Read[table +> schema |> mapping]would be preserved.