search

sunchao / parquet-rs

Apache Parquet implementation in Rust
Apache License 2.0
149 stars 20 forks source link

Update batch size in record reader #163

Closed sadikovi closed 5 years ago

sadikovi commented 5 years ago

This PR updates batch size from 256 to 1024. I also removed TODO for reader properties.

sadikovi commented 5 years ago

@sunchao could you review? Thanks!

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 622

Files with Coverage Reduction New Missed Lines %
encodings/encoding.rs 1 94.77%
file/properties.rs 10 92.55%
<!-- Total: 11 -->
Totals Coverage Status
Change from base Build 619: -0.008%
Covered Lines: 12414
Relevant Lines: 12993
💛 - Coveralls