[GitHub] [arrow-rs] alamb commented on a change in pull request #260: Fix empty Schema::metadata deserialization error

[GitBox <gi...@apache.org>]

alamb commented on a change in pull request #260:
URL: https://github.com/apache/arrow-rs/pull/260#discussion_r626650952



##########
File path: arrow/src/datatypes/schema.rs
##########
@@ -335,3 +336,35 @@ struct MetadataKeyValue {
     key: String,
     value: String,
 }
+
+#[cfg(test)]
+mod tests {
+    use crate::datatypes::DataType;
+
+    use super::*;
+
+    #[test]
+    fn test_ser_de_metadata() {

Review comment:
       In case anyone else is wondering, this test fails with the following error on master:
   
   ```
   
   ---- datatypes::schema::tests::test_ser_de_metadata stdout ----
   thread 'datatypes::schema::tests::test_ser_de_metadata' panicked at 'called `Result::unwrap()` on an `Err` value: Error("missing field `metadata`", line: 1, column: 284)', arrow/src/datatypes/schema.rs:355:53
   ```




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

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