alamb commented on code in PR #7741:
URL: https://github.com/apache/arrow-datafusion/pull/7741#discussion_r1346319872
##########
datafusion/physical-plan/src/aggregates/mod.rs:
##########
@@ -283,10 +283,6 @@ pub struct AggregateExec {
pub input: Arc<dyn ExecutionPlan>,
/// Schema after the aggregate is applied
schema: SchemaRef,
- /// Input schema before any aggregation is applied. For partial aggregate
this will be the
Review Comment:
It appears to have been added by @andygrove in
https://github.com/apache/arrow-datafusion/commit/b71e3ee5b7b85cea339d117e45731c5f9c7862db
/ https://github.com/apache/arrow/pull/9481
Since DataFusion has its own Serde of physical plans, I don't think we need
this anymore
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
