Skip to content

Commit

Permalink
PARQUET-895: Fix broken reading of nested repeated columns
Browse files Browse the repository at this point in the history
Use the column metadata to get the number of levels to process. It
fixes a problem where not enough values are returned in the case
of nested repeated values.

@peshopetrov should get the credit for 1st attempt

Closes apache#250

Change-Id: I1cc441557b8fb0819f7c6318a92d3dde998f329c
  • Loading branch information
mvertes authored and wesm committed Feb 23, 2017
1 parent 26dcd21 commit 86e2164
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/src/parquet/file/reader-internal.cc
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ std::unique_ptr<PageReader> SerializedRowGroup::GetColumnPageReader(int i) {
stream = properties_.GetStream(source_, col_start, col_length);

return std::unique_ptr<PageReader>(new SerializedPageReader(std::move(stream),
row_group_metadata_->num_rows(), col->compression(), properties_.memory_pool()));
col->num_values(), col->compression(), properties_.memory_pool()));
}

// ----------------------------------------------------------------------
Expand Down

0 comments on commit 86e2164

Please sign in to comment.