[kyuubi] branch master updated: [KYUUBI #5269][Bug] Dupllicated AnalyzesTablesCommand in kyuubi authz

[yao]

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 f967f2f8b [KYUUBI #5269][Bug] Dupllicated AnalyzesTablesCommand in 
kyuubi authz
f967f2f8b is described below

commit f967f2f8bff5fe18a9fc60f7306b4fb0e4bb1649
Author: Angerszhuuuu <angers....@gmail.com>
AuthorDate: Tue Sep 12 13:48:54 2023 +0800

    [KYUUBI #5269][Bug] Dupllicated AnalyzesTablesCommand in kyuubi authz
    
    ### _Why are the changes needed?_
    
    To close #5269 .
    
    1.  AnalyzesTablesCommand in table_command_spec  is duplicated and won't be 
processed
    2. The extractor desc of AnalyzesTablesCommand in table_command_spec is 
wrong
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run 
test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests)
 locally before make a pull request
    
    ### _Was this patch authored or co-authored using generative AI tooling?_
    
    Closes #5270 from AngersZhuuuu/KYUUBI-5269.
    
    Closes #5269
    
    75efc1adb [Angerszhuuuu] Update TableCommands.scala
    75a6fa223 [Angerszhuuuu] Update table_command_spec.json
    
    Authored-by: Angerszhuuuu <angers....@gmail.com>
    Signed-off-by: Kent Yao <y...@apache.org>
---
 .../src/main/resources/table_command_spec.json             | 14 --------------
 .../kyuubi/plugin/spark/authz/gen/TableCommands.scala      |  2 --
 2 files changed, 16 deletions(-)

diff --git 
a/extensions/spark/kyuubi-spark-authz/src/main/resources/table_command_spec.json
 
b/extensions/spark/kyuubi-spark-authz/src/main/resources/table_command_spec.json
index a9f2ec06e..0025fae75 100644
--- 
a/extensions/spark/kyuubi-spark-authz/src/main/resources/table_command_spec.json
+++ 
b/extensions/spark/kyuubi-spark-authz/src/main/resources/table_command_spec.json
@@ -908,20 +908,6 @@
   } ],
   "opType" : "ANALYZE_TABLE",
   "queryDescs" : [ ]
-}, {
-  "classname" : "org.apache.spark.sql.execution.command.AnalyzeTablesCommand",
-  "tableDescs" : [ {
-    "fieldName" : "tableIdent",
-    "fieldExtractor" : "TableIdentifierTableExtractor",
-    "columnDesc" : null,
-    "actionTypeDesc" : null,
-    "tableTypeDesc" : null,
-    "catalogDesc" : null,
-    "isInput" : true,
-    "setCurrentDatabaseIfMissing" : false
-  } ],
-  "opType" : "ANALYZE_TABLE",
-  "queryDescs" : [ ]
 }, {
   "classname" : "org.apache.spark.sql.execution.command.CacheTableCommand",
   "tableDescs" : [ ],
diff --git 
a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/gen/TableCommands.scala
 
b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/gen/TableCommands.scala
index b08169d39..f3754e4b9 100644
--- 
a/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/gen/TableCommands.scala
+++ 
b/extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/gen/TableCommands.scala
@@ -607,8 +607,6 @@ object TableCommands {
     AnalyzeColumn,
     AnalyzePartition,
     AnalyzeTable,
-    AnalyzeTable.copy(classname =
-      "org.apache.spark.sql.execution.command.AnalyzeTablesCommand"),
     AppendDataV2,
     CacheTable,
     CacheTableAsSelect,