leoyvens commented on PR #10876: URL: https://github.com/apache/datafusion/pull/10876#issuecomment-2167753769
Thanks @jonahgao for pointing out that shadowing case, I didn't know that was possible and the change really would have introduced a bug. I've added a sql test to make sure we don't regress there. But I did give another shot at correctly separating ordinary table refs from CTE refs. Let me know if the approach still has problems and there are other edges cases. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
