disallow on datasource table

Author: Davies Liu

sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala

@@ -453,12 +453,16 @@ case class AlterTableRecoverPartitionsCommand(
       throw new AnalysisException(
         s"Operation not allowed: ALTER TABLE RECOVER PARTITIONS on temporary tables: $tableName")
     }
+    if (DDLUtils.isDatasourceTable(table)) {
+      throw new AnalysisException(
+        s"Operation not allowed: ALTER TABLE RECOVER PARTITIONS on datasource tables: $tableName")
+    }
     if (table.tableType != CatalogTableType.EXTERNAL) {
       throw new AnalysisException(
         s"Operation not allowed: ALTER TABLE RECOVER PARTITIONS only works on external " +
           s"tables: $tableName")
     }
-    if (table.partitionColumnNames.isEmpty) {
+    if (DDLUtils.isTablePartitioned(table)) {
       throw new AnalysisException(
         s"Operation not allowed: ALTER TABLE RECOVER PARTITIONS only works on partitioned " +
           s"tables: $tableName")

sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala

@@ -409,11 +409,15 @@ case class RepairTableCommand(tableName: TableIdentifier) extends RunnableComman
       throw new AnalysisException(
         s"Operation not allowed: MSCK REPAIR TABLE on temporary tables: $tableName")
     }
+    if (DDLUtils.isDatasourceTable(table)) {
+      throw new AnalysisException(
+        s"Operation not allowed: MSCK REPAIR TABLE on datasource tables: $tableName")
+    }
     if (table.tableType != CatalogTableType.EXTERNAL) {
       throw new AnalysisException(
         s"Operation not allowed: MSCK REPAIR TABLE only works on external tables: $tableName")
     }
-    if (table.partitionColumnNames.isEmpty) {
+    if (DDLUtils.isTablePartitioned(table)) {
       throw new AnalysisException(
         s"Operation not allowed: MSCK REPAIR TABLE only works on partitioned tables: $tableName")
     }