-
Notifications
You must be signed in to change notification settings - Fork 3
Improve schemas-in-warehouse page for objects and arrays #1462
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for snowplow-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
6e7f3df to
a4865d1
Compare
| ``` | ||
|
|
||
| If the `"properties"` key is missing, the type for the entire object will be `STRING` instead of `RECORD`. | ||
| If the `"properties"` key is missing, the type for the entire object will be `JSON` instead of `RECORD`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wrote "properties" (in quotes) because that's how was written before this PR. And I extended that pattern elsewhere in this PR for consistency.
I'd happily change it simply properties (no quotes) if you prefer.
| } | ||
| ``` | ||
|
|
||
| If the `"properties"` key is missing, the type for the entire object will be `STRING` instead of `STRUCT`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Useful to add that the string is serialized as JSON (here and elsewhere)?
|
My most recent commit also fixed some mistakes regarding enums in BigQuery and Databricks. |

No description provided.