blueBlue0102 opened a new pull request, #901: URL: https://github.com/apache/yunikorn-k8shim/pull/901
### What is this PR for? The application state handling code in the UpdateApplication method of AsyncRMCallback struct (in scheduler_callback.go) could be improved. The current implementation uses a switch statement with a default case that actually handles two specific states, which may not align with the typical expectations for a default case. ### What type of PR is it? * [x] - Refactoring ### What is the Jira issue? [YUNIKORN-2835](https://issues.apache.org/jira/browse/YUNIKORN-2835) ### How should this be tested? This has already passed the CI tests(unit test) on my forked repo: [Link](https://github.com/blueBlue0102/yunikorn-k8shim/actions/runs/10551200123) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
