zanmato1984 opened a new issue, #40020:
URL: https://github.com/apache/arrow/issues/40020

   ### Describe the enhancement requested
   
   This is a followup of a discussion in the code review of 
https://github.com/apache/arrow/pull/39383/files#r1457036717. Back then I found 
row table related structures are using unsigned types as offset, which is 
inconsistent with the rest of the arrow project and causes some unnecessary 
static casts.
   
   Though I'm not exactly sure why the original author chose to use unsigned 
types as offset, I think we'd better change it to signed types if there is no 
substantial reasons about correctness.
   
   ### Component(s)
   
   C++


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