Minor: It is odd to me that `getBlocks` iterates through all the blocks and uses a map entry without a map, then this iterates through the blocks again. I think it would be less code and more straightforward if you iterated once with a counter and test the value of that counter against a set of indices.
[ Full content available at: https://github.com/apache/parquet-mr/pull/527 ] This message was relayed via gitbox.apache.org for [email protected]
