mqy commented on a change in pull request #9025:
URL: https://github.com/apache/arrow/pull/9025#discussion_r550173817



##########
File path: rust/arrow/src/datatypes.rs
##########
@@ -1903,9 +1929,20 @@ mod tests {
 
     #[test]
     fn serde_struct_type() {
+        let kv_array = [("k".to_string(), "v".to_string())];

Review comment:
       @nevi-me I just saw the comment "The JSON can either be an Object or an 
Array of Objects" from [ARROW-8883: [Rust] [Integration] Enable more 
tests](https://github.com/apache/arrow/commit/18b92818e7e12c6516511abe33b9e9ff78573571#diff-16170ebfd75c15cd0ef92a3e4f35dba353edcb11352101c363fc35cf3729267d)
   
   That's interesting! But I haven't seen the `Object` example from 
testing/data/arrow-ipc-stream/integration/1.0.0-littleendian/generated_custom_metadata.json.gz.
 So would you please give some pointers?




----------------------------------------------------------------
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:
[email protected]


Reply via email to