[jira] [Commented] (ARROW-5763) [JS] enable integration tests for MapVector

["Paul Taylor (JIRA)" <ji...@apache.org>]

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

Paul Taylor commented on ARROW-5763:
------------------------------------

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.

> [JS] enable integration tests for MapVector
> -------------------------------------------
>
>                 Key: ARROW-5763
>                 URL: https://issues.apache.org/jira/browse/ARROW-5763
>             Project: Apache Arrow
>          Issue Type: New Feature
>          Components: JavaScript
>            Reporter: Benjamin Kietzman
>            Priority: Minor
>
> As of 0.14, C++ and Java support Map arrays those implementations pass integration tests. JS has a MapVector and some unit tests for it, but it should be tested against other implementations as well



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