alamb commented on code in PR #17135:
URL: https://github.com/apache/datafusion/pull/17135#discussion_r2283465001


##########
datafusion/physical-plan/src/metrics/baseline.rs:
##########
@@ -173,15 +173,15 @@ impl SpillMetrics {
 #[derive(Debug, Clone)]
 pub struct SplitMetrics {
     /// Number of times an input [`RecordBatch`] was split
-    pub batches_splitted: Count,

Review Comment:
   i don't think we need a dedicated PR. We could add a note in the upgrade 
guide. 
   
   Given that these are such small changes, maybe just adding the `api-change` 
label so they are highlighted in the release notes would be good enough



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to