Re: [D] Deprecation question [arrow-rs]

["etseidl (via GitHub)" <gi...@apache.org>]

GitHub user etseidl closed a discussion: Deprecation question

I'm preparing to finally remove some functions deprecated back in v53, but a question cropped up with regards to `MetadataLoader` https://github.com/apache/arrow-rs/blob/721150286b00bece40ffcc6f5ac14ebb5d64785b/parquet/src/arrow/async_reader/metadata.rs#L89-L104

All of the functions in `MetadataLoader` were deprecated in v53.1.0, and so can be removed in v56.0.0...but `MetadataLoader` itself was not deprecated. Removing the deprecated functions basically leaves it as a zombie struct with no implementation.

My question is can we totally remove `MetadataLoader` at this point, or do we have to restart the timer by deprecating the struct and impl and wait until v59 or so?

@alamb 

GitHub link: https://github.com/apache/arrow-rs/discussions/7636

----
This is an automatically sent email for github@arrow.apache.org.
To unsubscribe, please send an email to: github-unsubscribe@arrow.apache.org