yashmayya opened a new pull request, #14075: URL: https://github.com/apache/pinot/pull/14075
- The `FROM_DATE_TIME` scalar function returns a long value representing milliseconds since epoch and is not a standard SQL function - https://github.com/apache/pinot/blob/49896eb3ba40ecc1494b1cc1783466c03aa7f832/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/DateTimeFunctions.java#L488 - The return type was forced to `TIMESTAMP` in the multistage engine in https://github.com/apache/pinot/pull/11350 and carried forward in the refactoring in https://github.com/apache/pinot/pull/13573. - This leads to an inconsistent experience for the same Pinot UDF across the two query engines:   - The `FROM_DATE_TIME` function only has a scalar implementation in Pinot (and no transform function equivalent) that returns a `long`, so it isn't too clear why the Calcite return type was forced to `TIMESTAMP`. -- 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]
