Julien Le Dem

@rdblue But typically object model integration (Avro, Thrift, Protobuf) is used to import data into Parquet or read data imported in the same model. Since people have existing schemas, it...

@rdblue I agree with your statement that we should not restrict reading a parquet file to only the model that wrote it however I don't think that this PR forces...

`java -jar parquet-benchmarks/target/parquet-benchmarks.jar org.apache.parquet.benchmarks.VectorizedConversionBenchmark.*.* -wi 2 -i 2 -f 8` ``` Benchmark Mode Samples Score Error Units o.a.p.b.VectorizedConversionBenchmark.t0_baseLine thrpt 16 2658.709 ± 69.008 ops/s o.a.p.b.VectorizedConversionBenchmark.t1_arrayInc thrpt 16 1372.189 ± 21.640...

@rdblue this looks good to go. Any other comments?

@swapnil: please create a PARQUET jira for this and prefix the description with the id: PARQUET-X: ... Also rebase your branch. Thank you. When this is done, I'll merge

@swapnilushinde gentle nagging on PRs is always fine :). Sometimes if your comment shows up at a busy time it falls through the cracks. Thank you for your contribution.

@swapnilushinde sorry your new PR is on the old repo. use apache/parquet-mr not Parquet/parquet-mr. (merging master in your branch is fine too since we'll squash in the end)

Is this fixed by https://github.com/apache/parquet-mr/pull/292 ?

@isnotinvain: should this be closed?

@smartkiwi what did you think of @rdblue's comment?