Add the checking logic in next commit and fix bug for changing commen…

…t of partition column

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

@@ -318,7 +318,7 @@ case class AlterTableChangeColumnCommand(
 
     // Find the origin column from dataSchema by column name.
     val originColumn = findColumnByName(table.dataSchema, columnName, resolver)
-    // Throw an AnalysisException if the column name/dataType is changed.
+    // Throw an AnalysisException if the column name is changed.
     if (!columnEqual(originColumn, newColumn, resolver)) {
       throw new AnalysisException(
         "ALTER TABLE CHANGE COLUMN is not supported for changing column " +
@@ -327,6 +327,16 @@ case class AlterTableChangeColumnCommand(
     }
 
     val typeChanged = originColumn.dataType != newColumn.dataType
+    val partitionColumnChanged = table.partitionColumnNames.contains(originColumn.name)
+
+    // Throw an AnalysisException if the type of partition column is changed.
+    if (typeChanged && partitionColumnChanged) {
+      throw new AnalysisException(
+        "ALTER TABLE CHANGE COLUMN is not supported for changing partition column" +
+          s"'${originColumn.name}' with type '${originColumn.dataType}' to " +
+          s"'${newColumn.name}' with type '${newColumn.dataType}'")
+    }
+
     val newDataSchema = table.dataSchema.fields.map { field =>
       if (field.name == originColumn.name) {
         // Add the comment to a column, if comment is empty, return the original column.

sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala

@@ -1702,6 +1702,11 @@ abstract class DDLSuite extends QueryTest with SQLTestUtils {
     sql("ALTER TABLE dbx.tab1 CHANGE COLUMN col1 col1 STRING")
     val column = catalog.getTableMetadata(tableIdent).schema.fields.find(_.name == "col1")
     assert(column.get.dataType == StringType)
+
+    // Ensure that changing partition column type throw exception
+    intercept[AnalysisException] {
+      sql("ALTER TABLE dbx.tab1 CHANGE COLUMN a a STRING")
+    }
   }
 
   test("drop build-in function") {