jorgecarleitao opened a new pull request #8260:
URL: https://github.com/apache/arrow/pull/8260


   This PR adds length to `LargeStringArray` and fixes undefined behavior.
   
   The undefined behavior was introduced in #7878. Specifically, we were 
(unsafely) interpreting a `i32` buffer as `u32`.
   
   This PR also changes the length kernel to return signed integers instead of 
unsigned ones. The reason for this logical change is that our offsets are 
stored as `i`, not `u`, and thus the kernel follows this convention (of using 
signed numbers to measure lengths).
   


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to