adriangb commented on code in PR #18719:
URL: https://github.com/apache/datafusion/pull/18719#discussion_r2533869125
##########
datafusion/physical-expr/src/projection.rs:
##########
@@ -396,6 +406,35 @@ impl ProjectionExprs {
))
}
+ /// Project a RecordBatch.
+ ///
+ /// This function accepts a pre-computed output schema instead of calling
[`ProjectionExprs::project_schema`]
+ /// so that repeated calls do not have schema projection overhead.
+ pub fn project_batch(
Review Comment:
Yep! I'm just thinking it would be less error prone to package it up in a
struct. I'll push the change here then we can rebase
https://github.com/apache/datafusion/pull/18627 to use the better version once
this is merged.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
