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

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/05/21 22:17:17 UTC

[GitHub] [spark] Kimahriman commented on pull request #36626: [SPARK-39249][SQL] Improve subexpression elimination for conditional expressions

Kimahriman commented on PR #36626:
URL: https://github.com/apache/spark/pull/36626#issuecomment-1133776367

   FYI I created https://github.com/apache/spark/pull/32987 a while ago to address this in a much more general way. I've tried to keep it up to date, but there seemed to be concerns about creating a subexpression for something that might only execute once I guess? Even though that's already happening in certain cases


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org