You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Wes McKinney (JIRA)" <ji...@apache.org> on 2017/05/08 21:55:04 UTC

[jira] [Commented] (ARROW-182) [C++] Remove Array::Validate virtual function and make a separate method

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

Wes McKinney commented on ARROW-182:
------------------------------------

I renamed this JIRA and marked it for 0.4.0. I would like to remove the {{Validate}} virtual from {{arrow::Array}} and instead implement the validation separately (similar to how {{Array::Equals}} is implemented now)

> [C++] Remove Array::Validate virtual function and make a separate method
> ------------------------------------------------------------------------
>
>                 Key: ARROW-182
>                 URL: https://issues.apache.org/jira/browse/ARROW-182
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Micah Kornfield
>            Priority: Minor
>             Fix For: 0.4.0
>
>
> We should Validate the bitmask is well formed for child arrays (all bits that don't correspond to a slot should be zero).    Once the validate method is added we should be able to make the parent classes method virtual to avoid new implementors missing it.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)