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

Micah Kornfield commented on ARROW-6929:
----------------------------------------

I'm not sure the condition checked ( "if (array.length() > 0 && array.offset() 
== 0 && array.value_offset(0) != 0)")
 return Status::Invalid("The first offset isn't zero");
 } at 
[https://github.com/apache/arrow/blob/master/cpp/src/arrow/array/validate.cc#L249]
 is a requirement for arrays ?

> [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: 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)

Reply via email to