dqkqd commented on code in PR #17926:
URL: https://github.com/apache/datafusion/pull/17926#discussion_r2404414956
##########
datafusion/expr/src/logical_plan/plan.rs:
##########
@@ -2543,16 +2543,22 @@ impl Window {
)
}
+ /// Create a new window function using the provided schema to avoid the
overhead of
+ /// building the schema again when the schema is already known.
+ ///
+ /// This method should only be called when you are absolutely sure that
the schema being
+ /// provided is correct for the window function. If in doubt, call
[try_new](Self::try_new) instead.
Review Comment:
I reuse the docs from `Aggregate::try_new_with_schema`
https://github.com/apache/datafusion/blob/9e8ec548fb6ef6bcfaa33abda67a17af6a8999e4/datafusion/expr/src/logical_plan/plan.rs#L3451-L3456
--
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]
