austin362667 opened a new issue, #6708:
URL: https://github.com/apache/arrow-rs/issues/6708

   **Describe the bug**
   
   The string array functions `length()` and `bit_length()` should not return 
`length` as signed integer.
   
   **To Reproduce**
   <!--
   Steps to reproduce the behavior:
   -->
   
   See 
https://github.com/apache/arrow-rs/blob/5ad621fd97032796f3da2d02948e65513de8c891/arrow-string/src/length.rs#L47
 and 
https://github.com/apache/arrow-rs/blob/5ad621fd97032796f3da2d02948e65513de8c891/arrow-string/src/length.rs#L111
   
   **Expected behavior**
   
   Should return unsigned integer, since there are no negative length in our 
context.
   
   **Additional context**
   
   Thanks for @alamb bringing this up in the discussion on 
https://github.com/apache/arrow-rs/pull/6671#discussion_r1826570314 
   
   Note: The actual [Arrow 
spec](https://arrow.apache.org/docs/cpp/compute.html#string-transforms) uses 
integer types.


-- 
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]

Reply via email to