yuchenhuo commented on a change in pull request #27642: [SPARK-30885][SQL] V1
table name should be fully qualified if catalog name is provided
URL: https://github.com/apache/spark/pull/27642#discussion_r382866793
##########
File path:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveSessionCatalog.scala
##########
@@ -396,7 +397,7 @@ class ResolveSessionCatalog(
}
case AnalyzeColumnStatement(tbl, columnNames, allColumns) =>
- val v1TableName = parseV1Table(tbl, "ANALYZE TABLE")
+ val v1TableName = parseTempViewOrV1Table(tbl, "ANALYZE TABLE")
Review comment:
Just want to check if I understand this correctly. This reason why we need
to make this change is that `parseV1Table` would accidentally add currentName
as the namespace for a temp view which is incorrect because temp view shouldn't
have a namespace?
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
