apache / apache/uniffle

[Improvement] Seprate code for the different roles in spark rssShuffleManager

Open
#1,602 0 comments 0 reactions 0 assignees View on GitHub
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?

For the current codebase, the different roles codes are all in the same `RssShuffleManager` , this is hard to read and understand for the new bies.

### How should we improve?

_No response_

### Are you willing to submit PR?

- [ ] Yes I am willing to submit a PR!

Contributor guide

Open the contributing guide

Research direction

Start by reading the current RssShuffleManager implementation and identify the different roles whose code is combined there. Trace how those roles are used before deciding how to separate them. Done means the roles are separated into clearer code while preserving the existing shuffle-manager behavior; the issue does not name specific tests to run.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, spark
Domain
backend, distributed-systems
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.