Jefffrey commented on code in PR #9019: URL: https://github.com/apache/arrow-rs/pull/9019#discussion_r2637188148
########## arrow-buffer/src/buffer/run.rs: ########## @@ -18,75 +18,107 @@ use crate::ArrowNativeType; use crate::buffer::ScalarBuffer; -/// A slice-able buffer of monotonically increasing, positive integers used to store run-ends +/// A slice-able buffer of monotonically increasing, positive integers used to +/// store run-ends. /// -/// # Logical vs Physical -/// -/// A [`RunEndBuffer`] is used to encode runs of the same value, the index of each run is -/// called the physical index. The logical index is then the corresponding index in the logical -/// run-encoded array, i.e. a single run of length `3`, would have the logical indices `0..3`. +/// Used to compactly represent runs of the same value. Values being represented Review Comment: Yup I ran into this issue earlier on; it makes sense to just hard link it, added 👍 -- 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]
