-
Notifications
You must be signed in to change notification settings - Fork 970
Support to get kyuubi server event with RESTful api #7232
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
base: master
Are you sure you want to change the base?
Conversation
b111637 to
c0428a1
Compare
c0428a1 to
e2c27a7
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #7232 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 696 697 +1
Lines 43530 43610 +80
Branches 5883 5893 +10
======================================
- Misses 43530 43610 +80 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| serverEvent.state, | ||
| serverEvent.serverIP, | ||
| serverEvent.serverConf.asJava, | ||
| serverEvent.serverEnv.asJava, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for a security perspective, we should redact sensitive info by default, maybe we can have a server side config to control the behavior, the candidate value can be ORIGINAL, REDACTED, NONE
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW, I remember there was a similar discussion about retrieving batch info
Why are the changes needed?
Support to get kyuubi server event with RESTful api
How was this patch tested?
UT.
Was this patch authored or co-authored using generative AI tooling?
No.