packyan commented on code in PR #2625:
URL: https://github.com/apache/incubator-kyuubi/pull/2625#discussion_r874322229


##########
extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/RangerSparkExtensionSuite.scala:
##########
@@ -273,6 +273,30 @@ abstract class RangerSparkExtensionSuite extends 
KyuubiFunSuite with SparkSessio
       doAs("admin", sql(s"DROP DATABASE IF EXISTS $db"))
     }
   }
+
+  test("show table extended") {

Review Comment:
   In PrivilegesBuilder, users should have SELECT privileges on a table to 
perform DESCRIBE operations on it, and `show table extended xxx` is similar to 
performing DESCRIBE operations on batched tables, so when it comes to `show 
tables extended xxx ` you should have SELECT privileges instead of USE 
privileges on these tables.
   
   Since the tables in the test case either have SELECT privileges or none at 
all, nothing changes. I will further improve the test case.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to