fix issue116: limit the reconnect times or duration #117
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…onnection times exceeds the connection numbers
AlexStocks
reviewed
Apr 17, 2024
AlexStocks
reviewed
Apr 17, 2024
|
总体改的挺好，把 import 格式恢复下 |
AlexStocks
reviewed
Apr 17, 2024
AlexStocks
changed the title
好的 import格式已经修改 编译器自己格式化了 :( |
AlexStocks
reviewed
Apr 17, 2024
|
检验一下逻辑，然后贴一下结果就行。 |
测试了一下，客户端的连接数设置 > 服务端可用连接数，抓包时间一分钟。使用现在的dubbo-getty1.4.9版本，TCP连接2421个，服务端TIME_WAIT 2584个。 |
AlexStocks
approved these changes
Apr 25, 2024
Lvnszn
approved these changes
Apr 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does:
1.客户端在read EOF时，停止重连维护连接池的行为
2.reconnect增加最大次数，当重连次数超过设定的连接数时，退出。
Which issue(s) this PR fixes:
Fixes #116
Special notes for your reviewer:
1.session增加了新的attribute（ignoreReconnectKey），用于标识是否重连
2.最大重连次数设定为用户设定的连接数
Does this PR introduce a user-facing change?: