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.

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

laplace_cdf: Presto throws while Velox returns non-NULL #10974

Open
kagamiori opened this issue Sep 11, 2024 · 1 comment
Open

laplace_cdf: Presto throws while Velox returns non-NULL #10974

kagamiori opened this issue Sep 11, 2024 · 1 comment

Comments

@kagamiori
Copy link
Contributor

Description

Query: select laplace_cdf(1, c0, 0.3) from (values nan()) t(c0);
Presto-0.289: Throws an exception "(INVALID_FUNCTION_ARGUMENT) laplaceCdf Function: scale must be greater than 0"
Velox: Returns NaN

Error Reproduction

No response

Relevant logs

No response

@kagamiori kagamiori added bug Something isn't working fuzzer-found labels Sep 11, 2024
@Yuhta
Copy link
Contributor

Yuhta commented Sep 12, 2024

Since NaN is defined to be > 0, the error message from Java is obviously wrong

@Yuhta Yuhta added presto-java-issue and removed bug Something isn't working labels Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants