[
https://issues.apache.org/jira/browse/ARROW-6929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17021529#comment-17021529
]
Wes McKinney commented on ARROW-6929:
-------------------------------------
Reviewing the changes, I agree that the first offset is not required to be
zero, but it is a guideline (and we shift everything to 0 in the C++ library
anyway). If the first offset is non-zero then you are sending more data than
you need to, so I think this validation check can be removed
> [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
> Assignee: Ben Kietzman
> Priority: Major
> Fix For: 0.16.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)
