Skip to content

Commit

Permalink
Parquet: Add placeholder for REE writing into Parquet
Browse files Browse the repository at this point in the history
To be implemented in later commits.
  • Loading branch information
zagto authored and felipecrv committed Jan 13, 2023
1 parent 73baf02 commit 5f8fcd1
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
4 changes: 4 additions & 0 deletions cpp/src/parquet/arrow/path_internal.cc
Original file line number Diff line number Diff line change
Expand Up @@ -801,6 +801,10 @@ class PathBuilder {
return Status::OK();
}

Status Visit(const ::arrow::RunEndEncodedArray& array) {
return Status::NotImplemented("Arrow REE array in Parquet");
}

Status Visit(const ::arrow::FixedSizeListArray& array) {
MaybeAddNullable(array);
int32_t list_size = array.list_type()->list_size();
Expand Down
1 change: 1 addition & 0 deletions cpp/src/parquet/column_writer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ struct ValueBufferSlicer {
NOT_IMPLEMENTED_VISIT(Struct);
NOT_IMPLEMENTED_VISIT(FixedSizeList);
NOT_IMPLEMENTED_VISIT(Dictionary);
NOT_IMPLEMENTED_VISIT(RunEndEncoded);
NOT_IMPLEMENTED_VISIT(Extension);

#undef NOT_IMPLEMENTED_VISIT
Expand Down

0 comments on commit 5f8fcd1

Please sign in to comment.