[jira] [Commented] (ARROW-6929) [C++] ValidateArray is out of sync with the ListArray IPC specification

["Antoine Pitrou (Jira)" <ji...@apache.org>]

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

Antoine Pitrou commented on ARROW-6929:
---------------------------------------

[~emkornfield@gmail.com] Can you check the current implementation? It's been significantly overhauled.

> [C++] ValidateArray is out of sync with the ListArray IPC specification
> -----------------------------------------------------------------------
>
>                 Key: ARROW-6929
>                 URL: https://issues.apache.org/jira/browse/ARROW-6929
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>            Reporter: Micah Kornfield
>            Priority: Major
>             Fix For: 1.0.0
>
>
> * It appears to check that null values take zero space
>  * It still checks for a begin offset of 0 if the array isn't sliced (technically this doesn't seem necessary and it could be non-zero even if the array wasn't sliced.)
>  * I think it also fails if an array is sliced to truncate it since it should compare length to data_extent instead of last_offset.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)