You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "guojidan (via GitHub)" <gi...@apache.org> on 2024/02/27 10:32:41 UTC

[I] rewrite `nvl/ifnull` to `nvl2` [arrow-datafusion]

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

                 if this pr is merged, we can rewrite `nvl/ifnull` function to `nvl2`, like:
   `nvl(expr1, expr2)` -> `nvl2(expr1, expr1, expr2)`
   
   🤔
   
   _Originally posted by @guojidan in https://github.com/apache/arrow-datafusion/issues/9364#issuecomment-1966243814_
               


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


Re: [I] rewrite `nvl/ifnull` to `nvl2` [arrow-datafusion]

Posted by "guojidan (via GitHub)" <gi...@apache.org>.
guojidan commented on issue #9365:
URL: https://github.com/apache/arrow-datafusion/issues/9365#issuecomment-1966249432

   take


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


Re: [I] rewrite `nvl/ifnull` to `nvl2` [datafusion]

Posted by "Jefffrey (via GitHub)" <gi...@apache.org>.
Jefffrey closed issue #9365: rewrite `nvl/ifnull` to `nvl2`
URL: https://github.com/apache/datafusion/issues/9365


-- 
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@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org


Re: [I] rewrite `nvl/ifnull` to `nvl2` [datafusion]

Posted by "Jefffrey (via GitHub)" <gi...@apache.org>.
Jefffrey commented on issue #9365:
URL: https://github.com/apache/datafusion/issues/9365#issuecomment-3415508373

   nvl has been rewritten to be a thin wrapper over coalesce now: #17991
   
   So I don't think this is needed anymore


-- 
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@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org