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