Phillip Cloud created PARQUET-1159:
--------------------------------------

             Summary: 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
             Fix For: cpp-1.4.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
(v6.4.14#64029)

Reply via email to