You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/05/19 06:45:58 UTC

[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #6941: Fix CAST transform function for chained transforms

Jackie-Jiang commented on pull request #6941:
URL: https://github.com/apache/incubator-pinot/pull/6941#issuecomment-843795027


   @yupeng9 Added more tests and find the issue where `cast double -> long -> int` is not the same as `cast double -> int` when the value is out of the int range. Fixed it by always cast first regardless of the type.


-- 
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.

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