HippoBaro commented on code in PR #9697:
URL: https://github.com/apache/arrow-rs/pull/9697#discussion_r3090919841
##########
parquet/src/util/push_buffers.rs:
##########
@@ -48,6 +59,13 @@ pub(crate) struct PushBuffers {
ranges: Vec<Range<u64>>,
/// The buffers of data that can be used to decode the Parquet file
buffers: Vec<Bytes>,
+ /// High-water mark set by [`Self::release_through`]. After a release,
+ /// no push, has_range, or read may target offsets below this value.
+ #[cfg(feature = "arrow")]
+ watermark: u64,
+ /// Whether `ranges`/`buffers` are sorted by range start.
+ /// Set to `false` on every `push_range`, restored lazily before reads.
+ sorted: bool,
Review Comment:
Latest version uses a Btree which takes care of the invariant for us,
thanks!
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
