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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
