apache / parquet-cpp / 1050

push

travis-ci

PARQUET-807: Allow user to retain ownership of parquet::FileMetaData.

Also implements PARQUET-808: opening file with existing metadata object.

This allows a user to create a reader only for the purposes of obtaining the metadata.

Do you all think it's worth having a convenience method for reading the metadata out of a file?

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes #213 from wesm/PARQUET-807 and squashes the following commits:

c1b5c7c [Wes McKinney] Use ReadMetaData function in test
d382cca [Wes McKinney] Add note about ARROW-455
05ecd37 [Wes McKinney] Implement/test opening with provided metadata. Do not close Arrow output files automatically
d790bb5 [Wes McKinney] Tweak
0dd4184 [Wes McKinney] Add ReadMetaData convenience method
97527ba [Wes McKinney] Change FileMetaData in ParquetFileReader to a shared_ptr so that ownership can be transferred away

35 of 35 new or added lines in 6 files covered. (100.0%)

10725 of 11517 relevant lines covered (93.12%)

141662.84 hits per line