viirya commented on code in PR #2000:
URL: https://github.com/apache/arrow-rs/pull/2000#discussion_r915198508
##########
arrow/src/array/builder/decimal_builder.rs:
##########
@@ -154,10 +166,59 @@ impl ArrayBuilder for DecimalBuilder {
}
}
+impl Decimal256Builder {
+ /// Creates a new `Decimal256Builder`, `capacity` is the number of bytes
in the values
+ /// array
+ pub fn new(capacity: usize, precision: usize, scale: usize) -> Self {
+ let values_builder = UInt8Builder::new(capacity);
+ let byte_width = 32;
+ Self {
+ builder: FixedSizeListBuilder::new(values_builder, byte_width),
+ precision,
+ scale,
+ }
+ }
+
+ /// Appends a byte slice into the builder.
+ ///
+ /// Automatically calls the `append` method to delimit the slice appended
in as a
+ /// distinct array element.
+ #[inline]
+ pub fn append_value(&mut self, value: &Decimal256) -> Result<()> {
+ let value_as_bytes = value.raw_value();
+
+ if self.builder.value_length() != value_as_bytes.len() as i32 {
Review Comment:
Good point. I think we should. It is a cheap check too so I think okay.
--
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]
