You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/04 05:46:36 UTC

[GitHub] [arrow-rs] jorgecarleitao commented on issue #1638: Question: Why are there 3 types of `OffsetSizeTrait`s?

jorgecarleitao commented on issue #1638:
URL: https://github.com/apache/arrow-rs/issues/1638#issuecomment-1116950756

   I introduced them. At the time BinaryArray, LargeBinaryArray, StringArray and LargeStringArray were 4 different structs. The `OffsetSize`s were introduced to simplify the code via generics. At the time I did not realize that it could be even simpler. I agree that it can be simplified (arrow2 [has only one](https://github.com/jorgecarleitao/arrow2/blob/main/src/types/offset.rs#L5)).
   


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org