xumingming commented on PR #58269: URL: https://github.com/apache/spark/pull/58269#issuecomment-6038845230
@cloud-fan Thanks for the reviews. Pushed c16bf7a531d addressing the review comments: - `CTERelationDef`'s doc no longer promises fail-fast: it now states the escaping-reference validation matches by exprId and is exact only for plans produced by a single analysis pass, with the residual stitched-plan gap (duplicated exprIds can let an escape slip through and bind to an unintended input) leaning on the `InlineCTE` validation comment for details. - The nested negative test comment now counts `m.a` as three scopes out of the deepest subquery and drops "nested-wide" for plain wording that the outer-scope scan also inspects nested subquery plans. -- 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]
