anoopj opened a new pull request, #3178:
URL: https://github.com/apache/iceberg-rust/pull/3178

   ## What changes are included in this PR?
   
   This is similar to https://github.com/apache/iceberg-rust/pull/3177 but for 
constant columns. 
   
   create_primitive_array_repeated built a constant boolean column with 
BooleanArray::from(vec![*value; n]), which allocates an n-byte Vec<bool> and 
then bit-packs it. Set or clear the bits directly with BooleanBuffer::new_set / 
new_unset instead. The sibling string and binary arms already skip the 
throwaway Vec via from_iter_values, so this just brings the bool arm in line.
   
   Same output, no nulls. This path materializes constant columns (partition 
values, defaults, metadata) on read, and boolean constants are uncommon, so 
it's a minor win.
   
   ## Are these changes tested?
   
   Covered by existing tests


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to