[ 
https://issues.apache.org/jira/browse/ARROW-5762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16887544#comment-16887544
 ] 

Paul Taylor commented on ARROW-5762:
------------------------------------

After reviewing the C++, the JS version of the Map type is not the same (it's 
essentially a Struct except instead of accessing child fields by field index, 
they're accessed by name). We should absolutely update the JS Map 
implementation before the 1.0 release.


> [Integration][JS] Integration Tests for Map Type
> ------------------------------------------------
>
>                 Key: ARROW-5762
>                 URL: https://issues.apache.org/jira/browse/ARROW-5762
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Integration, JavaScript
>            Reporter: Bryan Cutler
>            Priority: Major
>
> ARROW-1279 enabled integration tests for MapType between Java and C++, but 
> JavaScript had to be disabled for the map case due to an error.  Once this is 
> fixed, {{generate_map_case}} could be moved under {{generate_nested_case}} 
> with the other nested types.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to