8000 Change the STANDARD_MASK_SIZE calculation to use size of template type. by sebastiaan-dev · Pull Request #16807 · duckdb/duckdb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Change the STANDARD_MASK_SIZE calculation to use size of template type. #16807

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
Merged
merged 1 commit into from
Mar 25, 2025
Merged

Change the STANDARD_MASK_SIZE calculation to use size of template type. #16807

merged 1 commit into from
Mar 25, 2025

Conversation

sebastiaan-dev
Copy link
Contributor

Changed the calculation of STANDARD_MASK_SIZE of the TemplatedValidityMask class to be consistent with other operations based on the type of the template, instead of using the static typevalidity_t.

Tishj
Tishj approved these changes Mar 24, 2025
View reviewed changes
Copy link
Contributor
@Tishj Tishj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍

@Mytherin Mytherin merged commit 2c41e7c into duckdb:main Mar 25, 2025
48 checks passed
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 15, 2025
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@2c41e7c
8f9726f 
Change the STANDARD_MASK_SIZE calculation to use size of template type. (duckdb/duckdb#16807)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 15, 2025
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@2c41e7c
f038610 
Change the STANDARD_MASK_SIZE calculation to use size of template type. (duckdb/duckdb#16807)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 16, 2025
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@2c41e7c
4be013f 
Change the STANDARD_MASK_SIZE calculation to use size of template type. (duckdb/duckdb#16807)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 16, 2025
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@2c41e7c
b21ca59 
Change the STANDARD_MASK_SIZE calculation to use size of template type. (duckdb/duckdb#16807)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 17, 2025
@krlmlr
vendor: Update vendored sources to duckdb/duckdb@2c41e7c
8ca7d40 
Change the STANDARD_MASK_SIZE calculation to use size of template type. (duckdb/duckdb#16807)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tishj Tishj Tishj approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sebastiaan-dev @Tishj @Mytherin
0