alamb commented on code in PR #11237:
URL: https://github.com/apache/arrow-rs/pull/11237#discussion_r4218127127
##########
parquet/src/util/push_buffers.rs:
##########
@@ -165,19 +165,32 @@ impl PushBuffers {
/// Returns the index of a buffer that contains all bytes of `start..end`,
/// if any.
fn find(&self, start: u64, end: u64) -> Option<usize> {
- // Ranges can overlap. Thus, the buffer that starts nearest to `start`
- // is not always the buffer that contains `start..end`:
+ // Common case: the buffers do not overlap. Then only the last buffer
Review Comment:
thank yuo
--
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]
