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)