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

   ## Which issue does this PR close?
   
   Fixes https://github.com/apache/datafusion/issues/21066
   
   ## Rationale for this change
   
   The Unparser breaks in some certain scenarios, which also don't have test 
coverage.
   
   ## What changes are included in this PR?
   
     - Fix: dequalify outer SELECT columns when wrapping stacked projections as 
derived subqueries
     - Tests: parameterize unparser roundtrip tests over all dialects + 
optimized/unoptimized plans, add regression test suite
   
   ## Are these changes tested?
   
   Yes, they are tested, the majority of this PR is to introduce *more* test 
paths.
   
   ## Are there any user-facing changes?
   
   No breaking changes, but some SQL generated by the Unparser may end up being 
a little different


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

Reply via email to