You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/04/01 11:11:33 UTC

[GitHub] [arrow-datafusion] alamb opened a new issue, #5821: The type coercion is implemented as an optimizer - and it changes schemas of particular `Expr`essions.

alamb opened a new issue, #5821:
URL: https://github.com/apache/arrow-datafusion/issues/5821

                 The type coercion is implemented as an optimizer - and it changes schemas of particular `Expr`essions.
   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 https://github.com/apache/arrow-datafusion/pull/5734#discussion_r1153540699_
               


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] jackwener commented on issue #5821: Remove type coercion of case expression in Expr::Schema

Posted by "jackwener (via GitHub)" <gi...@apache.org>.

jackwener commented on issue #5821:
URL: https://github.com/apache/arrow-datafusion/issues/5821#issuecomment-1581798414

   I'm working on it


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] alamb closed issue #5821: Remove type coercion of case expression in Expr::Schema

Posted by "alamb (via GitHub)" <gi...@apache.org>.

alamb closed issue #5821: Remove type coercion of case expression in Expr::Schema
URL: https://github.com/apache/arrow-datafusion/issues/5821


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org