thswlsqls commented on PR #40071: URL: https://github.com/apache/shardingsphere/pull/40071#issuecomment-6116540981
@terrymanu Thanks for the review. You're right about the shadowing regression. Instead of dropping outer contexts during the merge, which would also hide columns that exist only in the farther table (such as an unqualified `status`), `ColumnSegmentBinder` now resolves an owner-qualified outer reference to the nearest context for that alias. `assertBindWithQuantifySubqueryReferencingShadowedOuterAlias` binds `x.user_id` and `status` inside the `ALL` subquery through the real binding path and gets `t_user` and `t_order`; the first fails on the previous head with `AmbiguousColumnException`. `assertBindWithOwnerFromShadowedOuterTable` covers the lookup itself. EXISTS and scalar subqueries share that lookup, so the same shadowing case now works there too. The test invocation is now on one line. -- 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]
