[GitHub] [arrow-datafusion] allenma commented on pull request #5911: Add if function support

["allenma (via GitHub)" <gi...@apache.org>]

allenma commented on PR #5911:
URL: https://github.com/apache/arrow-datafusion/pull/5911#issuecomment-1501758241

   Thanks @yjshen for your good suggestion, actually I thought before that implement the if function as syntax sugar of case when, but during implementation I found it is not easy do it in the logical plan -> physical plan phase, and need to do conversion in the syntax tree to logical plan phase, and implement it as build-in function is pretty straightforward.
   Anyway let me revisit the code again.


-- 
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