From d91a7c0f5b93bfb7061dcb6aa8b78dd31b7273b3 Mon Sep 17 00:00:00 2001 From: Andrew Lamb Date: Fri, 10 Jan 2025 22:09:39 -0500 Subject: [PATCH] Add comments to physical optimizer tests (#14075) --- .../tests/physical_optimizer/combine_partial_final_agg.rs | 4 ++++ .../physical_optimizer/limited_distinct_aggregation.rs | 6 ++++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/datafusion/core/tests/physical_optimizer/combine_partial_final_agg.rs b/datafusion/core/tests/physical_optimizer/combine_partial_final_agg.rs index 85076abdaf29..b8a96f0f5a22 100644 --- a/datafusion/core/tests/physical_optimizer/combine_partial_final_agg.rs +++ b/datafusion/core/tests/physical_optimizer/combine_partial_final_agg.rs @@ -15,6 +15,10 @@ // specific language governing permissions and limitations // under the License. +//! Tests for [`CombinePartialFinalAggregate`] physical optimizer rule +//! +//! Note these tests are not in the same module as the optimizer pass because +//! they rely on `ParquetExec` which is in the core crate. use std::sync::Arc; use arrow::datatypes::{DataType, Field, Schema, SchemaRef}; diff --git a/datafusion/core/tests/physical_optimizer/limited_distinct_aggregation.rs b/datafusion/core/tests/physical_optimizer/limited_distinct_aggregation.rs index 6910db6285a3..46a56fe1fb75 100644 --- a/datafusion/core/tests/physical_optimizer/limited_distinct_aggregation.rs +++ b/datafusion/core/tests/physical_optimizer/limited_distinct_aggregation.rs @@ -15,8 +15,10 @@ // specific language governing permissions and limitations // under the License. -//! Tests for the limited distinct aggregation optimizer rule - +//! Tests for [`LimitedDistinctAggregation`] physical optimizer rule +//! +//! Note these tests are not in the same module as the optimizer pass because +//! they rely on `ParquetExec` which is in the core crate. use super::test_util::{parquet_exec_with_sort, schema, trim_plan_display}; use std::sync::Arc;