You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/06/02 14:18:51 UTC
[GitHub] [arrow] anthonylouisbsb opened a new pull request, #13298: ARROW-17637: [C++][Gandiva] Fix TO_TIMESTAMP(INTEGER) function for big integer values
anthonylouisbsb opened a new pull request, #13298:
URL: https://github.com/apache/arrow/pull/13298
When to_timestamp function gets a big value, the function returns an incorrect date due integer overflow:
- TO_TIMESTAMP(1626255099[INT32]) -> '1969-12-14 04:54:53.816'
The correct response would be:
- TO_TIMESTAMP(1626255099[INT32]) -> '2021-07-14 09:31:39'
The error is because we cast the integers to timestamp(int64) after multiplying the result for the total of millis.
It is necessary to change the order and convert it to int64 before multiplying by the number of seconds
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] pravindra merged pull request #13298: ARROW-16738: [C++][Gandiva] Fix TO_TIMESTAMP(INTEGER) function for big integer values
Posted by GitBox <gi...@apache.org>.
pravindra merged PR #13298:
URL: https://github.com/apache/arrow/pull/13298
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #13298: ARROW-16738: [C++][Gandiva] Fix TO_TIMESTAMP(INTEGER) function for big integer values
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #13298:
URL: https://github.com/apache/arrow/pull/13298#issuecomment-1155118943
https://issues.apache.org/jira/browse/ARROW-16738
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #13298: ARROW-17637: [C++][Gandiva] Fix TO_TIMESTAMP(INTEGER) function for big integer values
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #13298:
URL: https://github.com/apache/arrow/pull/13298#issuecomment-1144922180
https://issues.apache.org/jira/browse/ARROW-17637
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org