alamb commented on PR #9113:
URL: 
https://github.com/apache/arrow-datafusion/pull/9113#issuecomment-1934169678

   > Should we include the package/feature in `datafusion-cli/Cargo.toml`?
   
   Since `array_expressions` is a default feature, these functions will (still) 
be in datafusion-cli. I double checked this:
   
   ```rust
   ❯ select array_to_string([1,2,3], '---');
   +---------------------------------------------------------------------+
   | array_to_string(make_array(Int64(1),Int64(2),Int64(3)),Utf8("---")) |
   +---------------------------------------------------------------------+
   | 1---2---3                                                           |
   +---------------------------------------------------------------------+
   1 row in set. Query took 0.006 seconds.
   ```
   
   > Wondering what will happen if user built the cli without array functions 
feature and tries to invoke array function in SQL
   
   They will get a "no such function" error. It may be worth considering a way 
to hint / add stub functions that could be registered that say something like 
"no such function, but there is a function named `array_to_string` in feature 
package array_functions" :thinking:


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