jecsand838 commented on code in PR #9291:
URL: https://github.com/apache/arrow-rs/pull/9291#discussion_r2806602623
##########
arrow-avro/src/reader/record.rs:
##########
@@ -867,6 +1111,56 @@ impl Decoder {
| Self::DurationMillisecond(values)
| Self::DurationMicrosecond(values)
| Self::DurationNanosecond(values) => values.push(buf.get_long()?),
+ #[cfg(feature = "avro_custom_types")]
+ Self::Int8(values) => values.push(buf.get_int()? as i8),
Review Comment:
Correct as well. I went ahead and updated this so that if the encoded Avro
value is outside the representable range of the Arrow target type, we now
return a decode error instead of silently producing an incorrect wrapped value.
--
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]
