devinjdangelo opened a new pull request, #10767:
URL: https://github.com/apache/datafusion/pull/10767

   ## Which issue does this PR close?
   
   closes #10664 
   
   ## Rationale for this change
   
   Queries involving window functions are common and should be supported for 
unparsing a plan into SQL.
   
   ## What changes are included in this PR?
   
   - Implements logic for unprojecting window function projections, similar to 
existing logic for aggregate functions
   - No longer throw unimplemented error on LogicalPlan::Window nodes
   - Fixes a subtle error in unparsing Window expressions
   
   ## Are these changes tested?
   
   Yes, with a new round trip plan_to_sql test
   
   ## Are there any user-facing changes?
   
   Additional queries can be unparsed
   


-- 
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