manikBS commented on PR #16910: URL: https://github.com/apache/pinot/pull/16910#issuecomment-3341165230
Hi @Jackie-Jiang, I noticed that in **JsonFunctions**, all the scalar functions ignore exceptions when the JSON object or JSONPath is invalid, and instead return the provided default value. However, in **JsonExtractScalarTransformFunction**, an exception is thrown in such cases. Should both follow a consistent behavior? If so, which approach would be preferable, returning the default value (like JsonFunctions) or throwing an exception? -- 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]
