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
Support submission timeout for yarn application manager and get the applicationInfo in-memory #4719
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4719 +/- ##
============================================
+ Coverage 58.03% 58.05% +0.01%
Complexity 13 13
============================================
Files 580 580
Lines 32270 32241 -29
Branches 4307 4303 -4
============================================
- Hits 18728 18717 -11
+ Misses 11739 11729 -10
+ Partials 1803 1795 -8
... and 15 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -104,7 +101,6 @@ class BatchJobSubmission( | |||
|
|||
override protected def currentApplicationInfo: Option[ApplicationInfo] = { |
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.
this function has side-effects, should add ()
kyuubi-server/src/main/scala/org/apache/kyuubi/operation/BatchJobSubmission.scala
Outdated
Show resolved
Hide resolved
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.
LGTM, please fill the PR description to mention the background
https://github.com/apache/kyuubi/actions/runs/4728413557/jobs/8389933543?pr=4719
|
always update do not fetch if not exists comments unknown side effects comments fix app start time refactor
…r and get the applicationInfo in-memory To prevent the create batch operation stuck. - [ ] 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 #4719 from turboFei/yarn_app_mgr. Closes #4719 1b0da93 [fwang12] check timeout Authored-by: fwang12 <fwang12@ebay.com> Signed-off-by: fwang12 <fwang12@ebay.com>
Why are the changes needed?
To prevent the create batch operation stuck.
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