You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@parquet.apache.org by "Uwe L. Korn (JIRA)" <ji...@apache.org> on 2018/06/09 19:05:00 UTC

[jira] [Updated] (PARQUET-1159) Compatibility with C++ iterators

     [ https://issues.apache.org/jira/browse/PARQUET-1159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Uwe L. Korn updated PARQUET-1159:
---------------------------------
    Fix Version/s:     (was: cpp-1.4.0)
                   cpp-1.5.0

> Compatibility with C++ iterators
> --------------------------------
>
>                 Key: PARQUET-1159
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1159
>             Project: Parquet
>          Issue Type: Bug
>          Components: parquet-cpp
>    Affects Versions: cpp-1.2.0
>            Reporter: Phillip Cloud
>            Assignee: Phillip Cloud
>            Priority: Major
>             Fix For: cpp-1.5.0
>
>
> There are some places where it would clean up the code quite a bit to use C++ STL iterators and be compatible with their APIs.
> Additionally, in this PR (https://github.com/apache/parquet-cpp/pull/403) I had to allocate a separate vector to hold byte swapped values, when what I really want to do is iterate over the existing values in reverse (starting at the last valid byte) so I don't have to copy them into a separate container. This can be done with a {{std::reverse_iterator}} which allows one to use the {{++}} operator everywhere.



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