[spark] branch master updated: [SPARK-34332][SQL][TEST][FOLLOWUP] Remove unnecessary test for ALTER NAMESPACE .. SET LOCATION

[wenchen]

This is an automated email from the ASF dual-hosted git repository.

wenchen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 12d3517  [SPARK-34332][SQL][TEST][FOLLOWUP] Remove unnecessary test 
for ALTER NAMESPACE .. SET LOCATION
12d3517 is described below

commit 12d351706b2ecdd4a3cf9dc8fa131e2412aad5b5
Author: Terry Kim <yumin...@gmail.com>
AuthorDate: Fri Dec 10 16:37:48 2021 +0800

    [SPARK-34332][SQL][TEST][FOLLOWUP] Remove unnecessary test for ALTER 
NAMESPACE .. SET LOCATION
    
    ### What changes were proposed in this pull request?
    
    This is a follow up to #34610 to remove unnecessary test for `ALTER 
NAMESPACE .. SET LOCATION`. The test being removed is already covered by 
https://github.com/apache/spark/blob/7b000116575cdc8e42653a694d9ad5273346fa61/sql/core/src/test/scala/org/apache/spark/sql/execution/command/AlterNamespaceSetLocationSuiteBase.scala#L65
    
    ### Why are the changes needed?
    
    To remove unnecessary test.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Removing test.
    
    Closes #34854 from imback82/followup-SPARK-34332.
    
    Authored-by: Terry Kim <yumin...@gmail.com>
    Signed-off-by: Wenchen Fan <wenc...@databricks.com>
---
 .../spark/sql/connector/DataSourceV2SQLSuite.scala       | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git 
a/sql/core/src/test/scala/org/apache/spark/sql/connector/DataSourceV2SQLSuite.scala
 
b/sql/core/src/test/scala/org/apache/spark/sql/connector/DataSourceV2SQLSuite.scala
index 44d7954..4c5a001 100644
--- 
a/sql/core/src/test/scala/org/apache/spark/sql/connector/DataSourceV2SQLSuite.scala
+++ 
b/sql/core/src/test/scala/org/apache/spark/sql/connector/DataSourceV2SQLSuite.scala
@@ -1272,22 +1272,6 @@ class DataSourceV2SQLSuite
     }
   }
 
-  test("ALTER NAMESPACE .. SET LOCATION using v2 catalog") {
-    withNamespace("testcat.ns1.ns2") {
-      sql("CREATE NAMESPACE IF NOT EXISTS testcat.ns1.ns2 COMMENT " +
-        "'test namespace' LOCATION '/tmp/ns_test_1'")
-      sql("ALTER NAMESPACE testcat.ns1.ns2 SET LOCATION '/tmp/ns_test_2'")
-      val descriptionDf = sql("DESCRIBE NAMESPACE EXTENDED testcat.ns1.ns2")
-      assert(descriptionDf.collect() === Seq(
-        Row("Namespace Name", "ns2"),
-        Row(SupportsNamespaces.PROP_COMMENT.capitalize, "test namespace"),
-        Row(SupportsNamespaces.PROP_LOCATION.capitalize, 
"file:/tmp/ns_test_2"),
-        Row(SupportsNamespaces.PROP_OWNER.capitalize, defaultUser),
-        Row("Properties", ""))
-      )
-    }
-  }
-
   private def testShowNamespaces(
       sqlText: String,
       expected: Seq[String]): Unit = {

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org