Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[DOCS] Add the document for Netty #1060

Closed
2 of 3 tasks
jerqi opened this issue Aug 1, 2023 · 2 comments
Closed
2 of 3 tasks

[DOCS] Add the document for Netty #1060

jerqi opened this issue Aug 1, 2023 · 2 comments
Assignees
@leixm
Labels
0.8 For 0.8 version release

Comments

@jerqi
Copy link
Contributor

jerqi commented Aug 1, 2023

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

Which parts of the documentation do you think need improvement?

We don't have the document to tell users how to use Netty to transfer data.

Affects Version(s)

master

Improving the documentation

server_guide.md
client_guide.md

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
@jerqi
Copy link
Contributor Author

jerqi commented Aug 1, 2023

@leixm Could you help me with this issue?

@leixm
Copy link
Contributor

leixm commented Aug 1, 2023

I will raise a PR later.

@jerqi jerqi added the 0.8 For 0.8 version release label Aug 1, 2023
leixm pushed a commit to leixm/incubator-uniffle that referenced this issue Aug 8, 2023
[apache#1060] Add the document for Netty.
7c92e8d
@jerqi jerqi mentioned this issue Aug 8, 2023
Merged
jerqi pushed a commit that referenced this issue Aug 8, 2023
@leixm
[#1060] doc: Add the document for Netty. (#1116)
ab91f7e 
### What changes were proposed in this pull request?
Add the document for Netty.

### Why are the changes needed?
For:  #1060

### Does this PR introduce _any_ user-facing change?
Yes.

Co-authored-by: leixianming <leixianming@didiglobal.com>
leixm pushed a commit to leixm/incubator-uniffle that referenced this issue Aug 8, 2023
[apache#1060] Fix typo.
5d9a0d2
@jerqi jerqi closed this as completed Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leixm leixm

Labels
0.8 For 0.8 version release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jerqi @leixm