-
Notifications
You must be signed in to change notification settings - Fork 85
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
Allow explicit cast between varlist CAST(var, varlist type) #2386
Conversation
4f27938
to
9a73d5e
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #2386 +/- ##
==========================================
+ Coverage 91.04% 91.24% +0.20%
==========================================
Files 1015 1022 +7
Lines 36012 36323 +311
==========================================
+ Hits 32788 33144 +356
+ Misses 3224 3179 -45 ☔ View full report in Codecov by Sentry. |
c143192
to
4d493f5
Compare
resultChildVector->setNull(i, inputChildVector->isNull(i)); | ||
if (!resultChildVector->isNull(i)) { | ||
// cast position i in child data vector | ||
OP_WRAPPER::template operation<OPERAND_TYPE, RESULT_TYPE, FUNC>( |
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.
This is covered
4d493f5
to
1b9ac93
Compare
6e8edde
to
43cee94
Compare
43cee94
to
2ac1809
Compare
Benchmark: (size of list between 0 to 40, select condition size(list) > 25)
ToDo:
add more tests for list of nested typesadd cast_error cases