Skip to content

Conversation

Mytherin
Copy link
Collaborator

@Mytherin Mytherin commented Feb 8, 2025

For the DictionaryDecoder - if we are executing a filter, we can execute the filter once on the dictionary. We can then figure out if a value is filtered out or not by looking at the filter result for each dictionary value.

@Mytherin
Copy link
Collaborator Author

Mytherin commented Feb 8, 2025

Actually I just realized a cool optimization that we could implement based on this - if the dictionary filters out all values, we don't even need to read the offset page and can skip it entirely.

@duckdb-draftbot duckdb-draftbot marked this pull request as draft February 8, 2025 18:52
@Mytherin Mytherin marked this pull request as ready for review February 8, 2025 21:46
@Mytherin Mytherin merged commit 2e88160 into duckdb:main Feb 9, 2025
49 checks passed
Antonov548 added a commit to Antonov548/duckdb-r that referenced this pull request Feb 27, 2025
Parquet reader: push table filters directly into dictionaries (duckdb/duckdb#16136)
krlmlr pushed a commit to duckdb/duckdb-r that referenced this pull request Mar 5, 2025
Parquet reader: push table filters directly into dictionaries (duckdb/duckdb#16136)
@Mytherin Mytherin deleted the columnreaderfilter branch April 2, 2025 09:25
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 15, 2025
Parquet reader: push table filters directly into dictionaries (duckdb/duckdb#16136)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 15, 2025
Parquet reader: push table filters directly into dictionaries (duckdb/duckdb#16136)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 17, 2025
Parquet reader: push table filters directly into dictionaries (duckdb/duckdb#16136)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 18, 2025
Parquet reader: push table filters directly into dictionaries (duckdb/duckdb#16136)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant