[KYUUBI #6973][AUTHZ] Support Paimon DELETE FROM / UPDATE / MERGE INTO commands #6974
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6974 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 689 689
Lines 42637 42637
Branches 5804 5804
======================================
Misses 42637 42637 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
# Conflicts: # extensions/spark/kyuubi-spark-authz/src/test/scala/org/apache/kyuubi/plugin/spark/authz/ranger/PaimonCatalogRangerSparkExtensionSuite.scala
davidyuan1223
changed the title
| @@ -31,6 +31,8 @@ import org.apache.kyuubi.util.AssertionUtils._ | |||
| @PaimonTest | |||
| class PaimonCatalogRangerSparkExtensionSuite extends RangerSparkExtensionSuite { | |||
| override protected val catalogImpl: String = "hive" | |||
| override protected val sqlExtensions: String = | |||
| "org.apache.paimon.spark.extensions.PaimonSparkSessionExtensions" | |||
There was a problem hiding this comment.
I remember this would break the Scala 2.13 UT, if so, you may want to enable this extension conditionally, or ask the Paimon community to handle Scala 2.13 properly.
There was a problem hiding this comment.
Really? I will check it after test workflow complete
There was a problem hiding this comment.
I remember this would break the Scala 2.13 UT, if so, you may want to enable this extension conditionally, or ask the Paimon community to handle Scala 2.13 properly.
you are right, i found paimon-spark profile has some questions, it only support spark3 with scala2.12, then the spark4 with scala2.13, i will jump the test case when scala version is 2.13, then i will submit an issue in paimon community
pan3793
changed the title
|
Thanks, merged to master |
Why are the changes needed?
Support ranger check with paimon Update & Delete & MergeInti Table Command
How was this patch tested?
Test Paimon Update & Delete Table Command with ranger check
#6973
Was this patch authored or co-authored using generative AI tooling?
No