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

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


The following commit(s) were added to refs/heads/master by this push:
     new 08b2a57798 [KYUUBI #6947] Test Dropping Columns
08b2a57798 is described below

commit 08b2a57798272a96059491213475bd70439fa485
Author: davidyuan <[email protected]>
AuthorDate: Wed Mar 12 10:53:51 2025 +0800

    [KYUUBI #6947] Test Dropping Columns
    
    ### Why are the changes needed?
    
    Ranger check with paimon missing check drop columns command
    #6947
    
    ### How was this patch tested?
    
    Test ranger check with drop columns commad
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No
    
    Closes #6952 from davidyuan1223/test_dropping_columns.
    
    Closes #6947
    
    27e7b3836 [davidyuan] Merge remote-tracking branch 'origin/master' into 
test_dropping_columns
    83815ef3b [davidyuan] test rename column name
    
    Authored-by: davidyuan <[email protected]>
    Signed-off-by: Kent Yao <[email protected]>
---
 .../PaimonCatalogRangerSparkExtensionSuite.scala       | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git 
a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/PaimonCatalogRangerSparkExtensionSuite.scala
 
b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/PaimonCatalogRangerSparkExtensionSuite.scala
index 83136ebefb..532f6450d7 100644
--- 
a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/PaimonCatalogRangerSparkExtensionSuite.scala
+++ 
b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/PaimonCatalogRangerSparkExtensionSuite.scala
@@ -441,6 +441,24 @@ class PaimonCatalogRangerSparkExtensionSuite extends 
RangerSparkExtensionSuite {
     }
   }
 
+  test("Dropping Columns") {
+    withCleanTmpResources(Seq(
+      (s"$catalogV2.$namespace1.$table1", "table"))) {
+      val createTable = createTableSql(namespace1, table1)
+      doAs(admin, sql(createTable))
+      val droppingColumnsSql =
+        s"""
+           |ALTER TABLE $catalogV2.$namespace1.$table1
+           |DROP COLUMNS (name)
+           |""".stripMargin
+
+      interceptEndsWith[AccessControlException] {
+        doAs(someone, sql(droppingColumnsSql))
+      }(s"does not have [alter] privilege on [$namespace1/$table1]")
+      doAs(admin, sql(droppingColumnsSql))
+    }
+  }
+
   def createTableSql(namespace: String, table: String): String =
     s"""
        |CREATE TABLE IF NOT EXISTS $catalogV2.$namespace.$table

Reply via email to