MaxGekk commented on a change in pull request #30929:
URL: https://github.com/apache/spark/pull/30929#discussion_r549088723
##########
File path:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableAddPartitionSuiteBase.scala
##########
@@ -22,6 +22,20 @@ import
org.apache.spark.sql.catalyst.analysis.PartitionsAlreadyExistException
import org.apache.spark.sql.catalyst.catalog.CatalogTypes.TablePartitionSpec
import org.apache.spark.sql.internal.SQLConf
+/**
+ * This base suite contains unified tests for the `ALTER TABLE .. ADD
PARTITION` command that
+ * check V1 and V2 table catalogs. The tests that cannot run for all supported
catalogs are
+ * located in more specific test suites:
+ *
+ * - V2 table catalog tests:
+ *
[[org.apache.spark.sql.execution.command.v2.AlterTableAddPartitionSuite]]
+ * - V1 table catalog tests:
+ *
[[org.apache.spark.sql.execution.command.v1.AlterTableAddPartitionSuiteBase]]
Review comment:
Thank you very much @HyukjinKwon . Without your comment, I would spend
much time in troubleshooting this.
----------------------------------------------------------------
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]
