feniljain commented on PR #19415: URL: https://github.com/apache/datafusion/pull/19415#issuecomment-3701952486
I was trying to write a detailed writeup on how I think this change would be made, and I think I now understand why you were confused by my initial observation. I must have read something wrong at that time, it is clear to me now. But this small exercise still leaves one doubt which still holds true, if I make a change in handling of args with `NULL` [here](https://github.com/apache/datafusion/blob/b818f93416d18d06374a0707f5ef571f8a384070/datafusion/functions-nested/src/utils.rs#L60-L71), that would mean making the same change for all of the functions I listed in [this](https://github.com/apache/datafusion/pull/19415#issuecomment-3677957617) message here. Is that okay with you? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
