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.

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

feat(net): poll (needs further tests) #751

Closed
wants to merge 11 commits into from

Conversation

TTaq
Copy link
Contributor

@TTaq TTaq commented Apr 22, 2024

No description provided.

@dragonosbot
Copy link

@TTaq: no appropriate reviewer found, use r? to override

@dragonosbot dragonosbot added the S-等待审查 Status: 等待assignee以及相关方的审查。 label Apr 22, 2024
@github-actions github-actions bot added the ambiguous The title of PR/issue doesn't match the format label Apr 22, 2024
@TTaq
Copy link
Contributor Author

TTaq commented Apr 22, 2024

#580

@TTaq
Copy link
Contributor Author

TTaq commented Apr 22, 2024

完成了poll的基本逻辑,但是还需要测试

@Chiichen
Copy link
Member

Chiichen commented Apr 22, 2024

请修改PR标题以符合conventional commit 格式

Please edit the tittle of this pull request to match conventional commit format

@fslongjin
Copy link
Member

该pr需要在@Saga1718 修复issue #743 之后,进行进一步的测试

@fslongjin fslongjin requested a review from GnoCiYeH April 22, 2024 09:23
@fslongjin fslongjin changed the title Poll dev wait test feat(network): poll (needs further tests) Apr 22, 2024
@github-actions github-actions bot added enhancement New feature or request and removed ambiguous The title of PR/issue doesn't match the format labels Apr 22, 2024
@Chiichen Chiichen changed the title feat(network): poll (needs further tests) feat(net): poll (needs further tests) Apr 23, 2024
TTaq and others added 4 commits May 16, 2024 15:02
第二个客户端连接服务端时无反应,且给服务端发送消息后会出现一个debug日志
debug日志的内容是segment not in receive window (35264002..35264011 not intersecting 35264011..35788290),will send challenge ACK
@fslongjin
Copy link
Member

这个pr的poll是忙等的,我将结合epoll,重新写一版。

@fslongjin fslongjin closed this Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request S-等待审查 Status: 等待assignee以及相关方的审查。
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants