-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](variant)fix variant type conflicts in nested types #52696
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
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
TPC-H: Total hot run time: 33668 ms |
TPC-DS: Total hot run time: 181615 ms |
ClickBench: Total hot run time: 29.25 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-H: Total hot run time: 33601 ms |
TPC-DS: Total hot run time: 184874 ms |
ClickBench: Total hot run time: 29.53 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-H: Total hot run time: 33291 ms |
TPC-DS: Total hot run time: 185430 ms |
ClickBench: Total hot run time: 29.88 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
| 2 {"nested":[{"a":1,"c":1.1}]} | ||
|
|
||
| -- !sql_1 -- | ||
| ["1", ""1""] |
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.
这个""1"" 看着不是一个合法的json输出
| // test cast to array<double> | ||
| qt_sql_14 """select cast(v['nested']['a'] as array<double>), size(cast(v['nested']['a'] as array<double>)) from ${table_name} order by k""" | ||
| qt_sql_15 """select cast(v['nested']['b'] as array<double>), size(cast(v['nested']['b'] as array<double>)) from ${table_name} order by k""" | ||
| qt_sql_16 """select cast(v['nested']['c'] as array<double>), size(cast(v['nested']['c'] as array<double>)) from ${table_name} order by k""" |
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.
in sql_select_batch
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-H: Total hot run time: 33338 ms |
TPC-DS: Total hot run time: 185550 ms |
ClickBench: Total hot run time: 29.78 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
eldenmoon
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
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
csun5285
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
1. fix variant type conflicts with scalar and array 2. fix multi-level array can be parse into variant which should be report as error 3. fix convert_field_to_type convert_numric_type do not use PrimitiveType 4. add UT for function_cast :: FromVariant
1. fix variant type conflicts with scalar and array 2. fix multi-level array can be parse into variant which should be report as error 3. fix convert_field_to_type convert_numric_type do not use PrimitiveType 4. add UT for function_cast :: FromVariant
1. fix variant type conflicts with scalar and array 2. fix multi-level array can be parse into variant which should be report as error 3. fix convert_field_to_type convert_numric_type do not use PrimitiveType 4. add UT for function_cast :: FromVariant
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)