[GitHub] [arrow-rs] viirya commented on a change in pull request #1476: Fix generate_non_canonical_map_case

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

viirya commented on a change in pull request #1476:
URL: https://github.com/apache/arrow-rs/pull/1476#discussion_r833773674



##########
File path: integration-testing/src/bin/arrow-json-integration-test.rs
##########
@@ -107,6 +109,47 @@ fn arrow_to_json(arrow_name: &str, json_name: &str, verbose: bool) -> Result<()>
     Ok(())
 }
 
+fn canonicalize_schema(schema: &Schema) -> Schema {

Review comment:
       The generated json has fields named other than "key" and 'value". I think this is why it is non canonical map case. But I'm not sure which arrow implementations can actually produce such map arrays.




-- 
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-unsubscribe@arrow.apache.org

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