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)