klion26 commented on code in PR #7557:
URL: https://github.com/apache/arrow-rs/pull/7557#discussion_r2156217089
##########
arrow-avro/src/codec.rs:
##########
@@ -192,6 +192,8 @@ pub enum Codec {
/// Represents Avro fixed type, maps to Arrow's FixedSizeBinary data type
/// The i32 parameter indicates the fixed binary size
Fixed(i32),
+ /// Represents Avro Uuid type, a FixedSizeBinary with a length of 16
Review Comment:
got it, thanks
##########
arrow-avro/src/reader/record.rs:
##########
@@ -531,4 +546,49 @@ mod tests {
assert_eq!(map_arr.len(), 1);
assert_eq!(map_arr.value_length(0), 0);
}
+
+ #[test]
+ fn test_fixed_decoding() {
Review Comment:
I'm not entirely sure about this; we could involve a maintainer for a double
check.
just noticed that we already have tests for other types.
--
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]
