search

brimdata / super

A novel data lake based on super-structured data
https://zed.brimdata.io/
BSD 3-Clause "New" or "Revised" License
1.39k stars 64 forks source link

ZNG type "error" should be an alias #1525

Closed philrz closed 3 years ago

philrz commented 4 years ago

ZNG type error is currently a Primitive Type in the ZNG spec, but we've reached consensus that it should be implemented as an alias in zq instead. Therefore the ZNG spec will need to have ZNG removed from the spec, then the zq implementation changed to match. #1524 should help make this possible.

When talking about making this change, we realized that there probably should be a place in the zq repo where we divulge the current list of aliased types and if there's any special handling attached to them. At the moment the only aliases are from the Zeek enum and port types, and this is covered in ZNG Compatibility with Zeek Logs. However once we turn error into an alias we probably should break out the proposed doc to describe all three there.

mccanne commented 3 years ago

Closing as we decided to keep "error" as a native type.