cloud-fan commented on a change in pull request #31637:
URL: https://github.com/apache/spark/pull/31637#discussion_r584458526
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
##########
@@ -150,6 +150,23 @@ trait CheckAnalysis extends PredicateHelper with
LookupCatalog {
case AlterTable(_, _, u: UnresolvedV2Relation, _) =>
failAnalysis(s"Table not found: ${u.originalNameParts.quoted}")
+ case command: V2PartitionCommand =>
+ command.table match {
+ case r @ ResolvedTable(_, _, table, _) => table match {
+ case t: SupportsPartitionManagement =>
+ if (t.partitionSchema.isEmpty) {
+ failAnalysis(s"Table ${r.name} is not partitioned.")
+ }
+ case _ =>
+ failAnalysis(s"Table ${r.name} does not support partition
management.")
+ }
+ case _ =>
+ }
+
+ // `ShowTableExtended` should have been converted to the v1 command if
the table is v1.
+ case _: ShowTableExtended =>
+ throw new AnalysisException("SHOW TABLE EXTENDED is not supported for
v2 tables.")
Review comment:
Yea we should. I only moved this one because it broke some tests.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
