Skip to content

Parquet: Remove deprecated VectorizedReader.setRowGroupInfo and ParquetValueReader.setPageSource - #12321

Merged
nastra merged 4 commits into
apache:mainfrom
ebyhr:ebi/parquet
Feb 20, 2025
Merged

nastra merged 4 commits into
apache:mainfrom
ebyhr:ebi/parquet

Conversation

@ebyhr

@ebyhr ebyhr commented Feb 18, 2025

Copy link
Copy Markdown
Member

No description provided.

@nastra

nastra commented Feb 19, 2025

Copy link
Copy Markdown
Contributor

since you're already handling iceberg-parquet deprecations, could you please include the deprecations around BaseParquetWriter and BaseParquetReaders?

@nastra

nastra commented Feb 19, 2025

Copy link
Copy Markdown
Contributor

@ebyhr the CI failures are unrelated and have been fixed on main. Can you rebase please, so that we can get this in?

@ebyhr

ebyhr commented Feb 19, 2025

Copy link
Copy Markdown
Member Author

@nastra Sure, rebased on main branch. Thanks for your review.

@nastra
nastra merged commit 43ef03b into apache:main Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants