[jira] [Comment Edited] (ARROW-5654) [C++] ChunkedArray should validate the types of the arrays

["Wes McKinney (JIRA)" <ji...@apache.org>]

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

Wes McKinney edited comment on ARROW-5654 at 6/19/19 8:34 PM:
--------------------------------------------------------------

Our policy has been that it's up to the developer-user to call {{Validate}} functions when they are concerned about sanitizing user input. That way applications that have other checks in place aren't paying the cost twice. So Validate() should be invoked at the Cython level


was (Author: wesmckinn):
Our policy has been that it's up to the developer-user to call {{Validate}} functions when they are concerned about sanitizing user input. That way applications that have other checks in place aren't paying the cost twice

> [C++] ChunkedArray should validate the types of the arrays
> ----------------------------------------------------------
>
>                 Key: ARROW-5654
>                 URL: https://issues.apache.org/jira/browse/ARROW-5654
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++, Python
>            Reporter: Joris Van den Bossche
>            Priority: Major
>             Fix For: 1.0.0
>
>
> Example from Python, showing that you can currently create a ChunkedArray with incompatible types:
> {code:python}
> In [8]: a1 = pa.array([1, 2])
> In [9]: a2 = pa.array(['a', 'b'])
> In [10]: pa.chunked_array([a1, a2])
> Out[10]:
> <pyarrow.lib.ChunkedArray object at 0x7fca50704d20>
> [
>   [
>     1,
>     2
>   ],
>   [
>     "a",
>     "b"
>   ]
> ]
> {code}
> So a {{ChunkedArray::Validate}} can be implemented (and which should probably be called by default upon creation?)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)