arrow

ARROW-5630: [C++][Parquet] Fix RecordReader accounting for repeated fields with non-nullable leaf

After chasing my tail a bit on this one, I found that for the case of repeated fields where the leaf node is non-nullable, the RecordReader was terminating without exhausting all data pages. The accounting logic for delimiting records was incrementing the "consumed" definition levels count correctly only in the "nullable values" case, but this must be done the same always when the definition level is greater than 0 (which holds for optional values as well as max repetition level > 0).

Closes #5395 from wesm/ARROW-5630-parquet-nonnullable-records and squashes the following commits:

436c235d1 <Wes McKinney> Push missing change
6a0438847 <Wes McKinney> Fix RecordReader accounting for repeated fields with non-nullable leaf

Authored-by: Wes McKinney <wesm+git@apache.org>
Signed-off-by: Benjamin Kietzman <bengilgit@gmail.com>