New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AUTHZ] Refactor Authz reflection with kyuubi-util's DynMethods #4873
Conversation
b5c547e
to
916b7af
Compare
Codecov Report
@@ Coverage Diff @@
## master #4873 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 556 556
Lines 30697 30677 -20
Branches 4000 3996 -4
======================================
+ Misses 30697 30677 -20
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
The lineage plugin would also benefit from this in follow-ups since it has similar reflection methods as Authz's. |
kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala
Show resolved
Hide resolved
Pending this PR. Local builds failed unexpectedly. Running
|
please add |
…t/ReflectUtils.scala
8441e66
to
95d4760
Compare
OK. Built and tested successfully by running with |
Thanks, merged to master |
…il's DynMethods ### _Why are the changes needed?_ - add reflection utils in kyuubi-util-scala, using kyuubi-util's DynMethods - continue to provided simplified reflection calling in scala ### _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 - [x] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request Closes apache#4873 from bowenliang123/authz-reflect. Closes apache#4873 d0a5084 [liangbowen] import 95d4760 [Cheng Pan] Update kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala 83e70f0 [liangbowen] authz reflect Lead-authored-by: liangbowen <liangbowen@gf.com.cn> Co-authored-by: Cheng Pan <pan3793@gmail.com> Signed-off-by: Cheng Pan <chengpan@apache.org>
Why are the changes needed?
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 locally before make a pull request