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

Phillip Cloud commented on PARQUET-1163:
----------------------------------------

It could be, though this particular issue came up because of the 
logical/physical type separation in parquet. I think in general we want to 
compare types when asserting that arrays are equal in tests, and leave the 
special case comparisons (like this one) as close to the location they are 
needed. However, if this is generally useful outside of parquet-cpp then I can 
move it to Arrow.

> [C++] Add test function to compare underlying values of Arrow arrays, not 
> including type
> ----------------------------------------------------------------------------------------
>
>                 Key: PARQUET-1163
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1163
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-cpp
>            Reporter: Phillip Cloud
>            Assignee: Phillip Cloud
>
> For unsigned integer typed Arrow arrays read in from parquet files we want to 
> compare the values (which are stored as signed integers) but we don't want to 
> fail an assertion because the types of the Arrow arrays are different.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to