alamb opened a new pull request, #16479: URL: https://github.com/apache/datafusion/pull/16479
(draft as it builds on #16430) ## Which issue does this PR close? - Follow on to https://github.com/apache/datafusion/pull/16430 from @suibianwanwank ## Rationale for this change While reviewing https://github.com/apache/datafusion/pull/16430 I spent some time reviewing the logic and arguments and wanted to encode what I found in comments ## What changes are included in this PR? Add doc comments to `AggregateWindowExpr::get_result_column` ## Are these changes tested? by CI ## Are there any user-facing changes? Just doc comments -- 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
