Skip to content

Commit 40f8a33

Browse files
committed
cargo fmt
1 parent 743ccc6 commit 40f8a33

2 files changed

Lines changed: 10 additions & 9 deletions

File tree

datafusion/core/tests/physical_optimizer/partition_statistics.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1059,11 +1059,13 @@ mod test {
10591059
};
10601060
assert_eq!(
10611061
expect_partial_stat,
1062-
*agg_partial.statistics_with_args(&StatisticsArgs::new().with_partition(Some(0)))?
1062+
*agg_partial
1063+
.statistics_with_args(&StatisticsArgs::new().with_partition(Some(0)))?
10631064
);
10641065
assert_eq!(
10651066
expect_partial_stat,
1066-
*agg_partial.statistics_with_args(&StatisticsArgs::new().with_partition(Some(1)))?
1067+
*agg_partial
1068+
.statistics_with_args(&StatisticsArgs::new().with_partition(Some(1)))?
10671069
);
10681070

10691071
let expect_partial_overall_stat = Statistics {

docs/source/library-user-guide/upgrading/55.0.0.md

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -112,13 +112,12 @@ Existing implementations of `partition_statistics` continue to work unchanged.
112112
The default `statistics_with_args` delegates to the deprecated method, so no
113113
migration is required until the deprecated method is removed.
114114

115-
> **Warning:** The delegation between the two methods is **one-way**. The default
116-
> `statistics_with_args` calls `partition_statistics`, but the default
117-
> `partition_statistics` does **not** call `statistics_with_args` — it returns
118-
> `Statistics::new_unknown`. So if you migrate a node to override **only**
119-
> `statistics_with_args`, any remaining caller of the deprecated
120-
> `partition_statistics` on that node will silently receive
121-
> `Statistics::new_unknown` instead of your real statistics.
115+
> **Warning:** The delegation is **one-way**: the default `statistics_with_args`
116+
> calls `partition_statistics`, but the default `partition_statistics` does
117+
> **not** call `statistics_with_args` — it returns `Statistics::new_unknown`.
118+
> Nodes that override only `statistics_with_args` will silently return
119+
> `Statistics::new_unknown` to any caller still using the deprecated
120+
> `partition_statistics`.
122121
123122
**Who is affected:**
124123

0 commit comments

Comments
 (0)