apache / arrow-rs

Official Rust implementation of Apache Arrow

https://arrow.apache.org/

Apache License 2.0

2.63k stars 803 forks source link

Add `ParquetError::NeedMoreData` mark `ParquetError` as `non_exhaustive` #6630

Closed etseidl closed 3 days ago

etseidl commented 1 month ago

Which issue does this PR close?

Part of #6447.

Rationale for this change

ParquetMetaDataReader currently uses ParquetError::IndexOutOfBound to indicate the need for more data. This is not the intended use for IndexOutOfBound.

What changes are included in this PR?

Adds a new enum value NeedMoreData.

Are there any user-facing changes?

Yes, this changes the ParquetError API as well as ParquetMetaDataReader.

alamb commented 3 days ago

Thank you for your help and patience @etseidl