-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
bugfix:fix key [transport.enableRmClientBatchSendRequest ] cache penetration #4678
Conversation
…tration if not configure in env or nacos seata.properties
…tration if not configure in env or nacos seata.properties
core/src/main/java/io/seata/core/rpc/netty/RmNettyRemotingClient.java
Outdated
Show resolved
Hide resolved
core/src/main/java/io/seata/core/rpc/netty/TmNettyRemotingClient.java
Outdated
Show resolved
Hide resolved
…tration if not configure in env or nacos seata.properties
Codecov Report
@@ Coverage Diff @@
## develop #4678 +/- ##
=============================================
- Coverage 48.44% 48.43% -0.01%
- Complexity 4035 4036 +1
=============================================
Files 735 735
Lines 25600 25612 +12
Branches 3161 3162 +1
=============================================
+ Hits 12401 12405 +4
- Misses 11860 11869 +9
+ Partials 1339 1338 -1
|
core/src/main/java/io/seata/core/rpc/netty/RmNettyRemotingClient.java
Outdated
Show resolved
Hide resolved
core/src/main/java/io/seata/core/rpc/netty/RmNettyRemotingClient.java
Outdated
Show resolved
Hide resolved
core/src/main/java/io/seata/core/rpc/netty/RmNettyRemotingClient.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
请把pr及作者信息按照已有登记示例,进行登记,并提交到此pr中
https://github.com/seata/seata/blob/develop/changes/zh-cn/develop.md
https://github.com/seata/seata/blob/develop/changes/en-us/develop.md
ok,已登记 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ⅰ. Describe what this PR did
fix key [transport.enableRmClientBatchSendRequest ] cache penetration if not configure in env or nacos seata.properties
Ⅱ. Does this pull request fix one issue?
fixes #4666
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews