parthchandra commented on issue #7040:
URL: https://github.com/apache/arrow-rs/issues/7040#issuecomment-2625273530

   The relevant part of the spec is here: 
https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#unsigned-integers
 
   Quoting from the above - 
   ```
   INT annotation can be used to specify unsigned integer types, along with a 
maximum number of bits in the stored value.
   ```
   and later 
   
   
   UINT_8 | IntType (bitWidth = 8, isSigned = false)
   -- | --
   
   I would interpret this to mean that only 8 bits are significant and that the 
bit pattern represents an unsigned value.
   I agree we should have consensus on what the correct interpretation should 
be, but perhaps we could use the parquet implementation as the guide. 
   Irrespective of what the parquet cli displays for the value, it certainly 
does not display a null, implying the value is valid and within the range of 
legal values. Wdyt? 


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