cetra3 commented on issue #22073: URL: https://github.com/apache/datafusion/issues/22073#issuecomment-4405503208
I think if we want postgresql semantics for nulls, then we should adjust the array udfs to internally sort out the null cases and then emit them accordingly. Or we could introduce a new UDF for `any` specifically, which was mentioned in the original PR. But I am torn as to whether we need them (postgresql null semantics), since the current udfs don't honour them and I don't know if there is a strong mandate in the rest of the code base. If the intent is to move towards them, and closer to postgres's behaviour maybe we should just adjust the existing udfs rather than introducing new behaviour equivalent ones -- 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]
