Jackie-Jiang commented on PR #11673: URL: https://github.com/apache/pinot/pull/11673#issuecomment-1736265936
> I guess to conserve the nullability. I don't have a strong preference, but some users have been asking to change the behavior. Maybe it is just better to keep the old behavior and to specify the null semantic in the documentation. I see. I think it is good to conserve the nullability. Since we have `"null"` as the default value, it should be fine to return the real `null` from the scalar function. Can you modify the code and remove `nullableParameters = true` -- 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]
