From 2776b343543e1ef60899dbe76ae79fff26a4e6b4 Mon Sep 17 00:00:00 2001 From: Enrico Minack Date: Thu, 31 Oct 2024 13:00:32 +0100 Subject: [PATCH] Merge require_sequenced_output and implicit_ordering --- cpp/src/arrow/dataset/scanner.cc | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/cpp/src/arrow/dataset/scanner.cc b/cpp/src/arrow/dataset/scanner.cc index 832e5d9160b91..9f480dc8b0e2b 100644 --- a/cpp/src/arrow/dataset/scanner.cc +++ b/cpp/src/arrow/dataset/scanner.cc @@ -1062,7 +1062,7 @@ Result MakeScanNode(acero::ExecPlan* plan, return batch; }); - auto ordering = require_sequenced_output ? Ordering::Implicit() : Ordering::Unordered(); + auto ordering = require_sequenced_output || implicit_ordering ? Ordering::Implicit() : Ordering::Unordered(); auto fields = scan_options->dataset_schema->fields(); if (scan_options->add_augmented_fields) { @@ -1071,11 +1071,6 @@ Result MakeScanNode(acero::ExecPlan* plan, } } - Ordering ordering = Ordering::Unordered(); - if (implicit_ordering) { - ordering = Ordering::Implicit(); - } - return acero::MakeExecNode( "source", plan, {}, acero::SourceNodeOptions{schema(std::move(fields)), std::move(gen), ordering});