nemanjapetr-db commented on PR #49351:
URL: https://github.com/apache/spark/pull/49351#issuecomment-2583331296

   Please take another look.
   
   CTERelationDef does not contain anchor any longer -- when needed it is 
fetched from its child via pattern matching. Code is greatly simplified and all 
previous convoluted questions on recursionAnchor are now moot.
   
   I added several exceptions for the unsupported cases of Union under the CTE 
Definition.
   
   Substitution rules for UnionLoop/Ref are added for 4 cases of Union under 
CTE Definition.
   
   CTERelationDef change has broken some tests, will work on those now.


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to