[GitHub] [arrow-datafusion] izveigor commented on pull request #7338: Support nulls and empty for array functions

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

izveigor commented on PR #7338:
URL: https://github.com/apache/arrow-datafusion/pull/7338#issuecomment-1696149093

   Hello, @jayzhan211!
   Sorry for the long delay. I will carefully review these changes tomorrow.
   
   However, I have some questions about the nature of functions in Arrow Datafusion. I recently thought about solving the issue: https://github.com/apache/arrow-datafusion/issues/6559 and I noticed a fact that User Defined Functions in DataFusion does not parameterize the null handling behaviour (Unlike DuckDB with `special` [link](https://duckdb.org/docs/api/python/function.html#null-handling)) So If we want to implement new array or other nested data types functions (like `struct_insert`) maybe it is better to remake signature's structure (I mean adding new boolean argument `special` to `Signature`).
   
   I want to hear @alamb's opinion about this opportunity.


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