yihua opened a new pull request, #11909:
URL: https://github.com/apache/arrow-rs/pull/11909

   # Which issue does this PR close?
   
   - Part of #11908
   
   # Rationale for this change
   
   The `FIXED_LEN_BYTE_ARRAY` decimal benchmarks only cover 16-byte, 
non-negative values, while parquet-mr based writers use the minimal width for 
the precision (7 bytes for `DECIMAL(15,2)`). This adds that case ahead of the 
change for #11908 so it can run on the benchmark runner.
   
   # What changes are included in this PR?
   
   Two groups read a `FIXED_LEN_BYTE_ARRAY(7)` `DECIMAL(15,2)` column with 
negative values, as `Decimal128` and, through a type hint, as `Decimal64`. The 
generator now writes the low `type_length` bytes of each value, which leaves 
the existing 16-byte data unchanged.
   
   # Are these changes tested?
   
   They are benchmarks.
   
   # Are there any significant user-facing changes?
   
   No.
   
   AI usage: written with an AI coding assistant under my direction; I reviewed 
the code and ran the benchmarks.
   


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