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

Reason: (0 vs. 0) IN list must not be empty #7507

Closed
laithsakka opened this issue Nov 10, 2023 · 1 comment
Closed

Reason: (0 vs. 0) IN list must not be empty #7507

laithsakka opened this issue Nov 10, 2023 · 1 comment
Assignees
@mbasmanova
Labels
bug Something isn't working fuzzer Issues related the to Velox fuzzer test components. fuzzer-found

Comments

@laithsakka
Copy link
Contributor

Description

https://app.circleci.com/pipelines/github/facebookincubator/velox/38090/workflows/7050e87b-2f0c-4e9e-b100-9b5d91f44d1c/jobs/256269/artifacts

Error Reproduction

NA

Relevant logs

No response
@laithsakka laithsakka added bug Something isn't working fuzzer Issues related the to Velox fuzzer test components. fuzzer-found labels Nov 10, 2023
@laithsakka laithsakka changed the title to triage. Reason: (0 vs. 0) IN list must not be empty Nov 10, 2023
@laithsakka
Copy link
Contributor Author

try {
VELOX_USER_CHECK_GT(size, 0, "IN list must not be empty");
} catch (...) {
return std::make_sharedexec::AlwaysFailingVectorFunction(
std::current_exception());
}

check does not exisit in GenericPredicate

@mbasmanova mbasmanova mentioned this issue Nov 10, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mbasmanova mbasmanova

Labels
bug Something isn't working fuzzer Issues related the to Velox fuzzer test components. fuzzer-found
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a check for empty IN list to IN Presto predicate mbasmanova/velox-1
2 participants
@laithsakka @mbasmanova