[GitHub] [pinot] nizarhejazi commented on pull request #8382: builtin datetime functions to avoid throwing NullPointerException when input value is null

[GitBox <gi...@apache.org>]

nizarhejazi commented on pull request #8382:
URL: https://github.com/apache/pinot/pull/8382#issuecomment-1075475131


   @Jackie-Jiang There is an issue w/ defining nullableParameters in the annotation that is applied at the method level (vs. at the parameter level). What if the scalar function has more than one parameter (e.g. dateTimeConvert) and the function cannot handle the case where only the main input parameter is null (the input column value), while it is perfectly valid for the other parameters to be nulls?


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org