-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
optimize: optimize AbstractLockManager#collectRowLocks logic #4103
Conversation
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, 请在changes文件夹中英两版1.5.0md中登记你的pr记录和作者信息
英文版直接按照标题填写登记即可,中文版可详细描述,比如 减少无需竞争锁时的内存占用 |
hello,欢迎你成为seata的贡献者,请将你的钉钉号发送至 364176773@qq.com 邮箱,我将会邀请你加入seata的贡献者队伍中 |
changes/en-us/1.5.0.md
Outdated
@@ -126,7 +126,7 @@ | |||
- [[#4034](https://github.com/seata/seata/pull/4034)] optimize Nacos, Consul, Zookeeper and Etcd3 configuration Junit test Class | |||
- [[#4055](https://github.com/seata/seata/pull/4055)] optimize NetUtil#getLocalAddress0 | |||
- [[#4056](https://github.com/seata/seata/pull/4056)] optimize the DurationUtil | |||
|
|||
- [[#4101](https://github.com/seata/seata/pull/4103)] optimize: optimize AbstractLockManager#collectRowLocks logic |
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.
4101->4103
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.
optimize:
这几个字符删掉,标题不用改
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.
+1
changes/1.5.0.md
Outdated
@@ -127,6 +127,7 @@ Seata 是一款开源的分布式事务解决方案,提供高性能和简单 | |||
- [[#4034](https://github.com/seata/seata/pull/4034)] 优化“优化 Nacos、Consul、Zookeeper、Etcd3 配置(#4019)”的单元测试类 | |||
- [[#4055](https://github.com/seata/seata/pull/4055)] 优化NetUtil的getLocalAddress0方法 | |||
- [[#4056](https://github.com/seata/seata/pull/4056)] 优化 DurationUtil | |||
- [[#4101](https://github.com/seata/seata/pull/4103)] 减少分支事务注册无需竞争锁时的内存占用 |
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.
4101->4103
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.
[[#4101] 内的4101改为4103
@@ -161,8 +162,7 @@ Seata 是一款开源的分布式事务解决方案,提供高性能和简单 | |||
- [dmego](https://github.com/dmego) | |||
- [zhixing](https://github.com/chenlei3641) | |||
- [siyu](https://github.com/Pinocchio2018) | |||
|
|||
|
|||
- [xujj](https://github.com/XBNGit) |
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.
后面留个空行
@@ -160,7 +160,7 @@ | |||
- [dmego](https://github.com/dmego) | |||
- [zhixing](https://github.com/chenlei3641) | |||
- [siyu](https://github.com/Pinocchio2018) | |||
|
|||
- [xujj](https://github.com/XBNGit) |
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.
后面留个空行
Codecov Report
@@ Coverage Diff @@
## develop #4103 +/- ##
==========================================
Coverage 49.41% 49.42%
- Complexity 3751 3753 +2
==========================================
Files 700 700
Lines 23614 23613 -1
Branches 2915 2914 -1
==========================================
+ Hits 11670 11671 +1
Misses 10751 10751
+ Partials 1193 1191 -2
|
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
Optimization: confirm that the list is created only when necessary to help GC, issue #4101
Ⅱ. Does this pull request fix one issue?
fixes #4101
Ⅲ. Why don't you add test cases (unit test/integration test)?
just the optimization of GC
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews