apache / apache/uniffle

[FEATURE] Implement reportShuffleResult with netty proto

Open
#2,522 2 comments 0 reactions 0 assignees View on GitHub
good first issue
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.

### Describe the feature

I observe the reportShuffleResult decoding in grpc will cost too much cpu time.

### Motivation

_No response_

### Describe the solution

_No response_

### Additional context

_No response_

### Are you willing to submit PR?

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

Contributor guide

Open the contributing guide

Research direction

Locate reportShuffleResult and compare its gRPC decoding path with the proposed Netty proto path. Read the surrounding shuffle-result RPC definitions and run relevant decoding or benchmark tests, if present. Done means the Netty proto path is implemented and report results remain correct while decoding CPU cost is reduced.

Written by the indexing model from the issue text.

Assessment

Tech stack
grpc, java
Domain
backend, distributed-systems, performance
Issue type
Feature
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.