yaooqinn commented on code in PR #5448:
URL: https://github.com/apache/kyuubi/pull/5448#discussion_r1364850426
##########
extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/HudiCatalogRangerSparkExtensionSuite.scala:
##########
@@ -371,4 +371,58 @@ class HudiCatalogRangerSparkExtensionSuite extends
RangerSparkExtensionSuite {
}
}
}
+
+ test("IndexBasedCommand") {
+ assume(
+ !isSparkV33OrGreater,
+ "Hudi index creation not supported on Spark 3.3 or greater currently")
+ withCleanTmpResources(Seq((s"$namespace1.$table1", "table"), (namespace1,
"database"))) {
+ doAs(admin, sql(s"CREATE DATABASE IF NOT EXISTS $namespace1"))
+ doAs(
+ admin,
+ sql(
+ s"""
+ |CREATE TABLE IF NOT EXISTS $namespace1.$table1(id int, name
string, city string)
+ |USING HUDI
+ |OPTIONS (
+ | type = 'cow',
+ | primaryKey = 'id',
+ | 'hoodie.datasource.hive_sync.enable' = 'false'
+ |)
+ |PARTITIONED BY(city)
+ |""".stripMargin))
+
+ // CreateIndexCommand
+ val createIndex = s"CREATE INDEX $index1 ON $namespace1.$table1 USING
LUCENE (id)"
Review Comment:
It's irrelevant to the implementation. According to the standard or common
practice, an index is an individual object apart from a database, table, etc.
There are code examples for how indexes are handled in ranger repo, we shall
consider follow that
--
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]
