Merge require_sequenced_output and implicit_ordering

apache · Oct 31, 2024 · 2776b34 · 2776b34
1 parent 0377ef9
commit 2776b34
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/cpp/src/arrow/dataset/scanner.cc b/cpp/src/arrow/dataset/scanner.cc
@@ -1062,7 +1062,7 @@ Result<acero::ExecNode*> 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<acero::ExecNode*> 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});