Replies: 2 comments
-
Thanks! I've addressed some of this in a recent PR. I'll address the rest shortly. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Okay, I've updated the spec a bit. I've tried to address the logical type feedback and some of the alias stuff as well: |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Aliases
I'd suggest to make the Aliases documentation a bit more clear.
Logical Types
My understanding here is that aliases can be used to construct logical types, but this is not very obvious with the current wording.
Also, the assignment of "timestamp64" to the alias keyword is mentioned but it doesn't appear in the code snippet.
By the way, the above become less of an issue when someone continues reading the spec and goes through the supported aliases.
I think a small editing and slight restructuring of the document will make it much more easier to consume.
Overall it's a great spec!
Beta Was this translation helpful? Give feedback.
All reactions