viirya commented on a change in pull request #27187: [SPARK-30497][SQL]
migrate DESCRIBE TABLE to the new framework
URL: https://github.com/apache/spark/pull/27187#discussion_r366169592
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
##########
@@ -759,6 +759,8 @@ class Analyzer(
u.failAnalysis(s"${ident.quoted} is a temp view not table.")
}
u
+ case u @ UnresolvedTableOrView(ident) =>
+ lookupTempView(ident).getOrElse(u)
Review comment:
Looks like it is very similar to `UnresolvedRelation`?
`UnresolvedRelation` can also be resolve to view by `ResolveTempViews`, can
be resolved to v2 relation by `ResolveTables`, can be resolved to v1 relation
by `ResolveRelations`.
Seems we have a few similar ways to do resolving?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
