alamb commented on issue #9261:
URL: 
https://github.com/apache/arrow-datafusion/issues/9261#issuecomment-1951821776

   TLDR I think implementing this function (perhaps call it `nvl` and add an 
alias to `ifnull`) would be a good addition. Thank you @guojidan 
   
   I thought DataFusion already supported the `NVL` function but I guess I was 
remembering the Nan equivalent of `nanvl`: 
https://arrow.apache.org/datafusion/user-guide/sql/scalar_functions.html#nanvl
   
   So the steps are something like:
   1. add function
   2. add sqllogictests
   3. add doc entries
   
   Should be a good feature 🏆 
   
   Can you please implement it as a ScalarUDF in 
https://github.com/apache/arrow-datafusion/tree/main/datafusion/functions/src/core
 (following the model of `nullif`) rather than adding to `BuiltInFunction`?
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to