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

zq update through "allow typedefs to be redefined" by mccanne #1526

Closed
wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link
Contributor

This is an auto-generated PR with a zq dependency update needing manual attention. brimdata/super#2386, authored by @mccanne, has been merged, however zq could not be updated automatically. Please see https:/brimsec/brim/actions/runs/677403237 for the original failing run. If a Brim update is needed, you may use the branch on this PR to do so.


allow typedefs to be redefined

This commit relaxes the constraint that type names cannot be redefined
to a different type. This follows the Z design principle of dealing
with whatever the user or client throws at the system as best we can.

The problem of schema versioning is outside the scope of the ZSON
format and can be solved at a higher layer with the appropriate Z
building blocks.

We also removed the end-of-sequence concept from the ZSON spec as
it is not relevant here as the definitions of type contexts and
redefinable types allows an implementation to create a seekable output.

Closes brimdata/super#1568

@philrz
Copy link
Contributor

philrz commented Mar 23, 2021

The CI passed on a subsequent zq commit which advanced the pointer, so this run may have failed due to an intermittent bug. #1529 has been opened to track that.

@philrz philrz closed this Mar 23, 2021
@nwt nwt deleted the zqbot-dynamic-typedefs-31bbcd0 branch April 1, 2021 21:54
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.

allow Z type names to be redefined
1 participant