[GitHub] [arrow] jorgecarleitao commented on pull request #9624: ARROW-11845: [Rust] Implement to_isize() for ArrowNativeTypes

2021-03-05 Thread GitBox
jorgecarleitao commented on pull request #9624: URL: https://github.com/apache/arrow/pull/9624#issuecomment-791864271 I agree @alamb : I think that this requires much more work because we do this in multiple places. @ericwburden thank you very much for your interest in arrow and

[GitHub] [arrow] jorgecarleitao commented on pull request #9624: ARROW-11845: [Rust] Implement to_isize() for ArrowNativeTypes

2021-03-04 Thread GitBox
jorgecarleitao commented on pull request #9624: URL: https://github.com/apache/arrow/pull/9624#issuecomment-790976766 fwiw, I think that there is a different way of approaching this. The only reason we are implementing to_isize/to_usize on NativeTpe is because we have a function to