Skip to content

Commit c525b57

Browse files
authored
MINOR: [C++] Remove unnecessary code in MultipathLevelBuilder::Write (apache#33647)
The PathBuilder run is unnecessary as `MultipathLevelBuilder::Make` on line 896 does the visitation already. Authored-by: Jin Shang <shangjin1997@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
1 parent 1a82720 commit c525b57

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

‎cpp/src/parquet/arrow/path_internal.cc‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -894,8 +894,6 @@ Status MultipathLevelBuilder::Write(const Array& array, bool array_field_nullabl
894894
MultipathLevelBuilder::CallbackFunction callback) {
895895
ARROW_ASSIGN_OR_RAISE(std::unique_ptr<MultipathLevelBuilder> builder,
896896
MultipathLevelBuilder::Make(array, array_field_nullable));
897-
PathBuilder constructor(array_field_nullable);
898-
RETURN_NOT_OK(VisitArrayInline(array, &constructor));
899897
for (int leaf_idx = 0; leaf_idx < builder->GetLeafCount(); leaf_idx++) {
900898
RETURN_NOT_OK(builder->Write(leaf_idx, context, callback));
901899
}

0 commit comments

Comments
 (0)