Avoid refs to the hive package

MaxGekk · MaxGekk · commit af4d508e4ae9 · 2020-12-26T17:33:49.000+03:00
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableAddPartitionSuiteBase.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableAddPartitionSuiteBase.scala
@@ -34,7 +34,7 @@ import org.apache.spark.sql.internal.SQLConf
  *     - V1 In-Memory catalog:
  *       [[org.apache.spark.sql.execution.command.v1.AlterTableAddPartitionSuite]]
  *     - V1 Hive External catalog:
- *       [[org.apache.spark.sql.hive.execution.command.AlterTableAddPartitionSuite]]
+ *       `org.apache.spark.sql.hive.execution.command.AlterTableAddPartitionSuite`
  */
 trait AlterTableAddPartitionSuiteBase extends QueryTest with DDLCommandTestUtils {
   override val command = "ALTER TABLE .. ADD PARTITION"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableDropPartitionSuiteBase.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableDropPartitionSuiteBase.scala
@@ -33,7 +33,7 @@ import org.apache.spark.sql.internal.SQLConf
  *     - V1 In-Memory catalog:
  *       [[org.apache.spark.sql.execution.command.v1.AlterTableDropPartitionSuite]]
  *     - V1 Hive External catalog:
- *       [[org.apache.spark.sql.hive.execution.command.AlterTableDropPartitionSuite]]
+ *       `org.apache.spark.sql.hive.execution.command.AlterTableDropPartitionSuite`
  */
 trait AlterTableDropPartitionSuiteBase extends QueryTest with DDLCommandTestUtils {
   override val command = "ALTER TABLE .. DROP PARTITION"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableRenamePartitionSuiteBase.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterTableRenamePartitionSuiteBase.scala
@@ -31,7 +31,7 @@ import org.apache.spark.sql.QueryTest
  *     - V1 In-Memory catalog:
  *       [[org.apache.spark.sql.execution.command.v1.AlterTableRenamePartitionSuite]]
  *     - V1 Hive External catalog:
- *       [[org.apache.spark.sql.hive.execution.command.AlterTableRenamePartitionSuite]]
+ *       `org.apache.spark.sql.hive.execution.command.AlterTableRenamePartitionSuite`
  */
 trait AlterTableRenamePartitionSuiteBase extends QueryTest with DDLCommandTestUtils {
   override val command = "ALTER TABLE .. RENAME PARTITION"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLCommandTestUtils.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLCommandTestUtils.scala
@@ -29,7 +29,7 @@ import org.apache.spark.sql.test.SQLTestUtils
  * is not applicable to all supported catalogs, it should be placed to a specific trait:
  *
  *   - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.CommandSuiteBase]]
- *   - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.CommandSuiteBase]]
+ *   - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.CommandSuiteBase`
  *   - V2 In-Memory catalog: [[org.apache.spark.sql.execution.command.v2.CommandSuiteBase]]
  */
 trait DDLCommandTestUtils extends SQLTestUtils {
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/DropTableSuiteBase.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/DropTableSuiteBase.scala
@@ -27,7 +27,7 @@ import org.apache.spark.sql.{AnalysisException, QueryTest, Row}
  *   - V2 table catalog tests: [[org.apache.spark.sql.execution.command.v2.DropTableSuite]]
  *   - V1 table catalog tests: [[org.apache.spark.sql.execution.command.v1.DropTableSuiteBase]]
  *     - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.DropTableSuite]]
- *     - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.DropTableSuite]]
+ *     - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.DropTableSuite`
  */
 trait DropTableSuiteBase extends QueryTest with DDLCommandTestUtils {
   override val command = "DROP TABLE"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/ShowPartitionsSuiteBase.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/ShowPartitionsSuiteBase.scala
@@ -30,7 +30,7 @@ import org.apache.spark.sql.types.{StringType, StructType}
  *   - V1 table catalog tests: [[org.apache.spark.sql.execution.command.v1.ShowPartitionsSuiteBase]]
  *     - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.ShowPartitionsSuite]]
  *     - V1 Hive External catalog:
- *       [[org.apache.spark.sql.hive.execution.command.ShowPartitionsSuite]]
+ *       `org.apache.spark.sql.hive.execution.command.ShowPartitionsSuite`
  */
 trait ShowPartitionsSuiteBase extends QueryTest with DDLCommandTestUtils {
   override val command = "SHOW PARTITIONS"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/ShowTablesSuiteBase.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/ShowTablesSuiteBase.scala
@@ -31,7 +31,7 @@ import org.apache.spark.sql.types.StructType
  *   - V2 table catalog tests: [[org.apache.spark.sql.execution.command.v2.ShowTablesSuite]]
  *   - V1 table catalog tests: [[org.apache.spark.sql.execution.command.v1.ShowTablesSuiteBase]]
  *     - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.ShowTablesSuite]]
- *     - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.ShowTablesSuite]]
+ *     - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.ShowTablesSuite`
  */
 trait ShowTablesSuiteBase extends QueryTest with DDLCommandTestUtils {
   override val command = "SHOW TABLES"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/AlterTableAddPartitionSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/AlterTableAddPartitionSuite.scala
@@ -29,7 +29,7 @@ import org.apache.spark.sql.execution.command
  *   - V1 In-Memory catalog:
  *     [[org.apache.spark.sql.execution.command.v1.AlterTableAddPartitionSuite]]
  *   - V1 Hive External catalog:
- *     [[org.apache.spark.sql.hive.execution.command.AlterTableAddPartitionSuite]]
+ *     `org.apache.spark.sql.hive.execution.command.AlterTableAddPartitionSuite`
  */
 trait AlterTableAddPartitionSuiteBase extends command.AlterTableAddPartitionSuiteBase {
   override protected def checkLocation(
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/AlterTableDropPartitionSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/AlterTableDropPartitionSuite.scala
@@ -28,7 +28,7 @@ import org.apache.spark.sql.execution.command
  *   - V1 In-Memory catalog:
  *     [[org.apache.spark.sql.execution.command.v1.AlterTableDropPartitionSuite]]
  *   - V1 Hive External catalog:
- *     [[org.apache.spark.sql.hive.execution.command.AlterTableDropPartitionSuite]]
+ *     `org.apache.spark.sql.hive.execution.command.AlterTableDropPartitionSuite`
  */
 trait AlterTableDropPartitionSuiteBase extends command.AlterTableDropPartitionSuiteBase {
   override protected val notFullPartitionSpecErr = "The following partitions not found in table"
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/AlterTableRenamePartitionSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/AlterTableRenamePartitionSuite.scala
@@ -30,7 +30,7 @@ import org.apache.spark.sql.internal.SQLConf
  *   - V1 In-Memory catalog:
  *     [[org.apache.spark.sql.execution.command.v1.AlterTableRenamePartitionSuite]]
  *   - V1 Hive External catalog:
- *     [[org.apache.spark.sql.hive.execution.command.AlterTableRenamePartitionSuite]]
+ *     `org.apache.spark.sql.hive.execution.command.AlterTableRenamePartitionSuite`
  */
 trait AlterTableRenamePartitionSuiteBase extends command.AlterTableRenamePartitionSuiteBase {
   protected def createSinglePartTable(t: String): Unit = {
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/DropTableSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/DropTableSuite.scala
@@ -25,7 +25,7 @@ import org.apache.spark.sql.execution.command
  * specific test suites:
  *
  *   - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.DropTableSuite]]
- *   - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.DropTableSuite]]
+ *   - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.DropTableSuite`
  */
 trait DropTableSuiteBase extends command.DropTableSuiteBase {
   test("purge option") {
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/ShowPartitionsSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/ShowPartitionsSuite.scala
@@ -26,7 +26,7 @@ import org.apache.spark.sql.execution.command
  * specific test suites:
  *
  *   - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.ShowPartitionsSuite]]
- *   - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.ShowPartitionsSuite]]
+ *   - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.ShowPartitionsSuite`
  */
 trait ShowPartitionsSuiteBase extends command.ShowPartitionsSuiteBase {
   test("show everything in the default database") {
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/ShowTablesSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/command/v1/ShowTablesSuite.scala
@@ -28,7 +28,7 @@ import org.apache.spark.sql.types.{BooleanType, StringType, StructType}
  * specific test suites:
  *
  *   - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.ShowTablesSuite]]
- *   - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.ShowTablesSuite]]
+ *   - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.ShowTablesSuite`
  */
 trait ShowTablesSuiteBase extends command.ShowTablesSuiteBase {
   override def defaultNamespace: Seq[String] = Seq("default")

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ import org.apache.spark.sql.test.SQLTestUtils`
`29`	`29`	`* is not applicable to all supported catalogs, it should be placed to a specific trait:`
`30`	`30`	`*`
`31`	`31`	`* - V1 In-Memory catalog: [[org.apache.spark.sql.execution.command.v1.CommandSuiteBase]]`
`32`		`- * - V1 Hive External catalog: [[org.apache.spark.sql.hive.execution.command.CommandSuiteBase]]`
	`32`	+ * - V1 Hive External catalog: `org.apache.spark.sql.hive.execution.command.CommandSuiteBase`
`33`	`33`	`* - V2 In-Memory catalog: [[org.apache.spark.sql.execution.command.v2.CommandSuiteBase]]`
`34`	`34`	`*/`
`35`	`35`	`trait DDLCommandTestUtils extends SQLTestUtils {`