[PARQUET-1159] [C++] Compatibility with C++ iterators

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Later
• Affects Version/s: cpp-1.2.0
• Fix Version/s: cpp-9.0.0
• Component/s: parquet-cpp
• Assignee: Unassigned
• Reporter: Phillip Cloud
• Created: 10/Nov/17 23:19
• Updated: 28/Jun/22 10:24
• Resolved: 28/Jun/22 10:24

Description

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.