veronica-m-ef commented on code in PR #7954:
URL: https://github.com/apache/arrow-rs/pull/7954#discussion_r2216802532


##########
arrow-avro/src/reader/record.rs:
##########
@@ -301,9 +301,23 @@ impl Decoder {
             }
             Codec::Uuid => Self::Uuid(Vec::with_capacity(DEFAULT_CAPACITY)),
         };
-        Ok(match data_type.nullability() {
-            Some(nullability) => Self::Nullable(
-                nullability,
+        let union_order = match data_type.nullability() {
+            None => None,
+            Some(Nullability::NullFirst) => Some(Nullability::NullFirst),
+            Some(Nullability::NullSecond) => {
+                if strict_mode {
+                    return Err(ArrowError::ParseError(
+                        "Found Avro union of the form ['T','null'], which is 
disallowed in strict_mode"
+                            .to_string(),
+                    ));
+                }
+                Some(Nullability::NullSecond)
+            }
+        };

Review Comment:
   @jecsand838 thank you! All done now :)



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to