You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
alamb
changed the title
The type coercion is implemented as an optimizer - and it changes schemas of particular Expressions.
Remove type coercion of case expression in Expr::Schema
Apr 1, 2023
As part of fixing #5733 in #5734 we installed a workaround that manipulates the calculation of the type of a Case expression
A better solution would be to fix the expression initially (by coercing the types correctly), but that was blocked on #5733
Once #5733 is complete, we should remove the workaround put in Expr::schema
Details below:
And it takes care of
Expr::Case
.But
Expr::Projection
caches the schema in itself - so it is not updated! Therefore we get a mismatch between declared schema vs batch schema.Essentially this PR makes
Expr::Case
to not change a schema during the coercion optimizer.Originally posted by @mslapek in #5734 (comment)
The text was updated successfully, but these errors were encountered: