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

Antoine Pitrou updated PARQUET-1159:
------------------------------------
    Fix Version/s:     (was: cpp-5.0.0)
                   cpp-6.0.0

> [C++] 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-6.0.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
(v8.3.4#803005)

Reply via email to