alamb commented on code in PR #8080:
URL: https://github.com/apache/arrow-rs/pull/8080#discussion_r2263347463
##########
parquet/src/errors.rs:
##########
@@ -52,6 +52,9 @@ pub enum ParquetError {
/// Returned when a function needs more data to complete properly. The
`usize` field indicates
/// the total number of bytes required, not the number of additional bytes.
NeedMoreData(usize),
+ /// Returned when a function needs more data to complete properly.
Review Comment:
FWIW I could remove `ParquetError::NeedMoreDataRange` if I reworked the
decoder to have a proper state machine internally (rather than calling into the
existing `ParquetMetaDataDecoder`).
That is certainly my long term plan. If we like this API I will invest some
time into seeing if I can sketch out what it would look like
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
