Skip to content

Conversation

@etseidl
Copy link
Contributor

@etseidl etseidl commented Jun 27, 2025

Which issue does this PR close?

Part of #7810

Rationale for this change

Reducing code bloat.

What changes are included in this PR?

Remove deprecated functions and structs.

Are these changes tested?

Removed functions were not referenced so no tests necessary

Are there any user-facing changes?

Yes, public functions are removed.

@etseidl etseidl added parquet Changes to the parquet crate api-change Changes to the arrow API labels Jun 27, 2025
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Let's do it

@etseidl
Copy link
Contributor Author

etseidl commented Jun 27, 2025

Thanks for the fast review @alamb!

@etseidl etseidl merged commit 3183e03 into apache:main Jun 27, 2025
16 checks passed
@etseidl etseidl deleted the remove_parquet_deprecations branch June 27, 2025 22:22
@alamb
Copy link
Contributor

alamb commented Jun 28, 2025

🧹

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

Labels

api-change Changes to the arrow API parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants