Skip to content

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Apr 1, 2025

Which issue does this PR close?

Rationale for this change

It is not clear from the documentation why the parquet crate has no row group / data page predicate evaluation features as described on #7348

What changes are included in this PR?

  1. Document the rationale of why the parquet implementation does not include predicate evaluation (and instead has the building blocks to do so)

Are there any user-facing changes?

Docs only, no functional change

@alamb alamb added the documentation Improvements or additions to documentation label Apr 1, 2025
@github-actions github-actions bot added the parquet Changes to the parquet crate label Apr 1, 2025
Copy link
Member

@westonpace westonpace left a comment

Choose a reason for hiding this comment

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

Minor nits

Copy link
Contributor

@etseidl etseidl left a comment

Choose a reason for hiding this comment

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

❤️

Co-authored-by: Weston Pace <[email protected]>
Co-authored-by: Kyle Barron <[email protected]>
@alamb
Copy link
Contributor Author

alamb commented Apr 3, 2025

Thank you all for correcting what seems to be pretty atrocious typos on my part

@alamb alamb merged commit fec03ea into apache:main Apr 4, 2025
16 checks passed
@alamb alamb deleted the alamb/clarify_pushdown branch April 4, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pushdown predictions to Parquet in-memory row group fetches

4 participants