JigaoLuo commented on issue #7910: URL: https://github.com/apache/arrow-rs/issues/7910#issuecomment-3065107407
Thanks for the confirmation! I also agree that `remaining_bytes` should be changed to u64, since these fields representing **file** offsets—and shouldn't be affected by differences in 32-bit OR 64-bit memory addressing space. I’m happy to submit a PR to update both offset and `remaining_bytes` to u64. Do you happen to see any other questionable uses of usize that might need changing as well? -- 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
