[Improvement] Close coordinator client when DelegationRSSShuffleManager fallback to SortShuffleManager
- Dominant language
- Java
- Stars
- 454
- Forks
- 172
- Avg merge
- 5d 17h
- Merged PRs (30d)
- 5
Description
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
### Search before asking
- [X] I have searched in the [issues](https://github.com/apache/incubator-uniffle/issues?q=is%3Aissue) and found no similar issues.
### What would you like to be improved?
When `DelegationRSSShuffleManager` falls back to `SortShuffleManager`, the coordinator clients is not closed.
The clients should be closed when RSS is not used.
### How should we improve?
_No response_
### Are you willing to submit PR?
- [X] Yes I am willing to submit a PR!
Contributor guide
Research direction
Start by tracing the fallback path in DelegationRSSShuffleManager when it switches to SortShuffleManager. Confirm which coordinator clients were created and ensure they are closed when RSS is not used; done means no coordinator clients remain open after fallback.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- distributed-systems
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100