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

Check data chunk for invisible records before evaluating expressions #14163

Open

kwannoel opened this issue Dec 23, 2023 · 1 comment

Labels

no-issue-activity type/feature

Contributor

kwannoel commented Dec 23, 2023

#13890 (comment)

The reason is that some expressions ignore the visibility and return non-null values for invisible rows. Examples include literal expression and simd-optimized expressions such as =. The CASE expression doesn't mask invisible rows in the condition result, so later values may override earlier ones.

Continuing on from this, for our data chunks, we should check if they contain any invisible records, before running any expressions on it.

kwannoel added the type/feature label

github-actions bot added this to the release-1.6 milestone

kwannoel removed this from the release-1.6 milestone

Contributor

github-actions bot commented Jun 12, 2024

This issue has been open for 60 days with no activity. Could you please update the status? Feel free to continue discussion or close as not planned.

github-actions bot added the no-issue-activity label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment