-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
implement nested identifier access #12614
Conversation
This reverts commit 00a12bc.
I didn't realize that there is already a function called get_field which has similar usage to struct_extract 😅 |
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.
Thank you @Lordworms -- this is really nice 😍
Really nice way to reuse get_field
} AS complex_data; | ||
|
||
query T | ||
SELECT complex_data.user.info.personal.name FROM complex_view; |
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.
😍
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.
🚀
Thanks @Lordworms |
Which issue does this PR close?
Closes #11445
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?