celeborn
celeborn copied to clipboard
[CELEBORN-1421] Refine code in master to reduce unnecessary sync to get workers/lostworkers/shutdownWorkers.
What changes were proposed in this pull request?
- Use ConcurrentSet to replace ArrayList for workers.
- Remove unnecessary sync and snapshot when get workers/lostworkers/shutdownWorkers
Why are the changes needed?
- Reduce unnecessary sync to get workers/lostworkers/shutdownWorkers.
- Somewhere in the Master, directly using statusSystem.workers(ArrayList) is not safe, potentially leading to concurrent modification issues.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GA