aokolnychyi commented on PR #36303: URL: https://github.com/apache/spark/pull/36303#issuecomment-1105745203
Alright, I think we will need both. - PR #36216 does not solve the problem in `identifyRootFields` and tests from this PR fail. - This PR does not address the problem of conflicting names, which is solved by PR #36216. After this PR, the output returned by `PushDownUtils$pruneColumns` will only include columns from one relation. However, we still apply `ProjectionOverSchema` on filters with subqueries that may reference other relations too. That's why we need both changes. @allisonwang-db, what do you think? -- 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]
