alamb opened a new issue, #9371: URL: https://github.com/apache/arrow-rs/issues/9371
We found some inconsistency in the arrow-rs codebase Some places use `*view as i32` and some places use `*view as u32`. We use a consistent practice and document it. I think most other places use `u32` (rather than `i32`) -- for example https://github.com/apache/arrow-rs/blob/6505d2a354e87e193e00ca7c6ea945de92c7adca/arrow-array/src/array/byte_view_array.rs#L372-L371 https://github.com/apache/arrow-rs/blob/6505d2a354e87e193e00ca7c6ea945de92c7adca/arrow-array/src/array/byte_view_array.rs#L339-L338 _Originally posted by @alamb in https://github.com/apache/arrow-rs/pull/9363#discussion_r2771409372_ -- 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]
