Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cast(double as real) behaves differently from Presto on a corner case #5934

Open
kagamiori opened this issue Jul 31, 2023 · 0 comments
Open

Cast(double as real) behaves differently from Presto on a corner case #5934

kagamiori opened this issue Jul 31, 2023 · 0 comments
Labels
bug Something isn't working triage Newly created issue that needs attention.

Comments

@kagamiori
Copy link
Contributor

Bug description

In Presto,

SELECT cast(1.7E308 as real); -- Infinity
SELECT cast(-1.7E308 as real); -- -Infinity

In Velox, both query throws. We should make them behave the same as Presto.

System information

Tested via presto CLI on devserver.

Relevant logs

No response
@kagamiori kagamiori added bug Something isn't working triage Newly created issue that needs attention. labels Jul 31, 2023
@bikramSingh91 bikramSingh91 mentioned this issue Aug 2, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working triage Newly created issue that needs attention.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix cast-double-to-real when input is beyond the limit kagamiori/velox
1 participant
@kagamiori