[jira] [Commented] (ARROW-3236) [C++] OutputStream bookkeeping logic when writing IPC file format is incorrect

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

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

Wes McKinney commented on ARROW-3236:
-------------------------------------

So the reason this bug was never caught is that the first message is the schema, which is redundant with the schema in the footer. We never read this first message in the C++ reader library. Patch incoming to make code more robust

> [C++] OutputStream bookkeeping logic when writing IPC file format is incorrect
> ------------------------------------------------------------------------------
>
>                 Key: ARROW-3236
>                 URL: https://issues.apache.org/jira/browse/ARROW-3236
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>    Affects Versions: 0.10.0
>            Reporter: Wes McKinney
>            Assignee: Wes McKinney
>            Priority: Blocker
>             Fix For: 0.11.0
>
>
> The {{position_}} field is initialized as -1, but the file writer class begins writing and aligning the format without updating the position, so this results in an extra byte written. This was reported in https://github.com/apache/arrow/issues/2559. A bit disturbing that this was not caught in unit tests



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