kyuubi icon indicating copy to clipboard operation
kyuubi copied to clipboard

[KYUUBI #7079] Improve performance of AccessRequest initialization

Open wForget opened this issue 5 months ago • 1 comments
trafficstars

Why are the changes needed?

Improve performance of authz rules.

Constantize some properties to reduce object creation:

  • AccessRequest initialization requires reflection methods
  • ~~RuleHelper.ugi~~ (different sparkSession may have different authz ugi)

closes #7079

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

No

wForget avatar Jun 05 '25 05:06 wForget

Codecov Report

Attention: Patch coverage is 0% with 57 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (cad5a39) to head (bb469c7). Report is 21 commits behind head on master.

Files with missing lines Patch % Lines
...uubi/plugin/spark/authz/ranger/AccessRequest.scala 0.00% 57 Missing :warning:
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #7081    +/-   ##
=======================================
  Coverage    0.00%   0.00%            
=======================================
  Files         697     700     +3     
  Lines       43203   43457   +254     
  Branches     5854    5887    +33     
=======================================
- Misses      43203   43457   +254     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Jun 05 '25 10:06 codecov-commenter