[SPARK-32512][SQL][TESTS][FOLLOWUP] Remove duplicate tests for ALTER TABLE .. PARTITIONS from DataSourceV2SQLSuite

[MaxGekk]

What changes were proposed in this pull request?

Remove tests from DataSourceV2SQLSuite that were copied to AlterTablePartitionV2SQLSuite by #29339.

Why are the changes needed?

• To reduce tests execution time
• To improve test maintenance

Does this PR introduce any user-facing change?

No

How was this patch tested?

By running the modified tests:

$ build/sbt "test:testOnly *DataSourceV2SQLSuite"
$ build/sbt "test:testOnly *AlterTablePartitionV2SQLSuite"

[MaxGekk]

spark/sql/core/src/test/scala/org/apache/spark/sql/connector/AlterTablePartitionV2SQLSuite.scala

Lines 32 to 41 in 1eb236b

	test("ALTER TABLE RECOVER PARTITIONS") {
	val t = "testcat.ns1.ns2.tbl"
	withTable(t) {
	spark.sql(s"CREATE TABLE $t (id bigint, data string) USING foo")
	val e = intercept[AnalysisException] {
	sql(s"ALTER TABLE $t RECOVER PARTITIONS")
	}
	assert(e.message.contains("ALTER TABLE RECOVER PARTITIONS is only supported with v1 tables"))
	}
	}

[MaxGekk]

spark/sql/core/src/test/scala/org/apache/spark/sql/connector/AlterTablePartitionV2SQLSuite.scala

Lines 43 to 57 in 1eb236b

	test("ALTER TABLE ADD PARTITION") {
	val t = "testpart.ns1.ns2.tbl"
	withTable(t) {
	spark.sql(s"CREATE TABLE $t (id bigint, data string) USING foo PARTITIONED BY (id)")
	spark.sql(s"ALTER TABLE $t ADD PARTITION (id=1) LOCATION 'loc'")
	val partTable = catalog("testpart").asTableCatalog
	.loadTable(Identifier.of(Array("ns1", "ns2"), "tbl")).asInstanceOf[InMemoryPartitionTable]
	assert(partTable.partitionExists(InternalRow.fromSeq(Seq(1))))
	val partMetadata = partTable.loadPartitionMetadata(InternalRow.fromSeq(Seq(1)))
	assert(partMetadata.containsKey("location"))
	assert(partMetadata.get("location") == "loc")
	}
	}

[MaxGekk]

spark/sql/core/src/test/scala/org/apache/spark/sql/connector/AlterTablePartitionV2SQLSuite.scala

Lines 103 to 112 in 1eb236b

	test("ALTER TABLE RENAME PARTITION") {
	val t = "testcat.ns1.ns2.tbl"
	withTable(t) {
	spark.sql(s"CREATE TABLE $t (id bigint, data string) USING foo PARTITIONED BY (id)")
	val e = intercept[AnalysisException] {
	sql(s"ALTER TABLE $t PARTITION (id=1) RENAME TO PARTITION (id=2)")
	}
	assert(e.message.contains("ALTER TABLE RENAME PARTITION is only supported with v1 tables"))
	}
	}

[MaxGekk]

spark/sql/core/src/test/scala/org/apache/spark/sql/connector/AlterTablePartitionV2SQLSuite.scala

Lines 114 to 125 in 1eb236b

	test("ALTER TABLE DROP PARTITION") {
	val t = "testpart.ns1.ns2.tbl"
	withTable(t) {
	spark.sql(s"CREATE TABLE $t (id bigint, data string) USING foo PARTITIONED BY (id)")
	spark.sql(s"ALTER TABLE $t ADD PARTITION (id=1) LOCATION 'loc'")
	spark.sql(s"ALTER TABLE $t DROP PARTITION (id=1)")
	val partTable =
	catalog("testpart").asTableCatalog.loadTable(Identifier.of(Array("ns1", "ns2"), "tbl"))
	assert(!partTable.asPartitionable.partitionExists(InternalRow.fromSeq(Seq(1))))
	}
	}

[SparkQA]

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36028/

[cloud-fan]

GA passed, merging to master, thanks!

[SparkQA]

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36028/

[SparkQA]

Test build #131422 has finished for PR 30444 at commit c2141d8.

• This patch fails SparkR unit tests.
• This patch merges cleanly.
• This patch adds no public classes.