I think this class needs more documentation. It isn't clear what these factory 
methods do without looking closely at the implementation. For example, 
single(rowCount) doesn't tell me that the resulting range is [0, rowCount). 
Similarly, it isn't clear what this is doing. Why is it called "build" when it 
is a factory method and not a builder?

[ Full content available at: https://github.com/apache/parquet-mr/pull/527 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to