-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[Improve][Transform] Improve inner sql query for map field type #7718
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
Conversation
| c_row.c_string c_string, | ||
| c_row.c_inner_row.c_inner_string c_inner_string, | ||
| c_row.c_inner_row.c_inner_timestamp c_inner_timestamp, | ||
| c_row.c_inner_row.c_map inner_map, |
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.
Please update Assert config.
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.
Done
Hisoka-X
left a comment
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.
Please update doc.
1f1d5c8 to
4085e87
Compare
60ef6d9 to
866866d
Compare
Hisoka-X
left a comment
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.
LGTM. Thanks @liunaijie

Purpose of this pull request
Now the sql transform is support inner query, we can use sql like this
But for now, if the
sub_col_bis map type. will got exception.improve this feature to support query map type as latest field.
Why query map type got exception:
when parse the sql, we need return the query column's field type.
if the query column is map, get the column type is too complex. so only support query from map type only once.
Does this PR introduce any user-facing change?
no
How was this patch tested?
update e2e test conf
Check list
New License Guide
release-note.