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

optimize: use hget replace hmget because only one field #4761

Merged
merged 10 commits into from Aug 1, 2022
Merged

optimize: use hget replace hmget because only one field #4761

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
028465e
Optimize: use hget replace hmget because only one field
conghuhu Jul 9, 2022
05c91fa
Merge branch 'develop' into optimize-redislock
conghuhu Jul 13, 2022
5f4f639
Merge branch 'develop' into optimize-redislock
conghuhu Jul 23, 2022
e2e885e
Merge branch 'develop' into optimize-redislock
conghuhu Jul 26, 2022
ba95c12
Merge branch 'develop' into optimize-redislock
funky-eyes Jul 31, 2022
5237252
optimize: add copyright notice (#4800)
AlexStocks Jul 26, 2022
a559082
bugfix: fix in high version springboot property not resolved (#4817)
liujunlin5168 Jul 28, 2022
6b2e762
optimize: automatically publish once a day (#4819)
wangliang181230 Jul 29, 2022
e1c9dc6
doc: add #4761 PR
conghuhu Jul 31, 2022
5238ca0
Merge branch 'optimize-redislock' of github.com:conghuhu/seata into o…
conghuhu Jul 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions changes/en-us/develop.md
View file
Open in desktop
Expand Up @@ -16,6 +16,7 @@ Add changes here for all PR submitted to the develop branch.
- [[#4750](https://github.com/seata/seata/pull/4750)] optimize unBranchLock romove xid
- [[#4797](https://github.com/seata/seata/pull/4797)] optimize the github actions
- [[#4800](https://github.com/seata/seata/pull/4800)] Add NOTICE as Apache License V2
- [[#4761](https://github.com/seata/seata/pull/4761)] use hget replace hmget because only one field

### test:
- [[#4794](https://github.com/seata/seata/pull/4794)] try to fix the test `DataSourceProxyTest.getResourceIdTest()`
Expand Down
1 change: 1 addition & 0 deletions changes/zh-cn/develop.md
View file
Open in desktop
Expand Up @@ -16,6 +16,7 @@
- [[#4765](https://github.com/seata/seata/pull/4765)] mysql8.0.29版本及以上XA模式不持connection至二阶段
- [[#4797](https://github.com/seata/seata/pull/4797)] 优化所有github actions脚本
- [[#4800](https://github.com/seata/seata/pull/4800)] 按照 Apache 协议规范，添加 NOTICE 文件
- [[#4761](https://github.com/seata/seata/pull/4761)] 使用 hget 代替 RedisLocker 中的 hmget, 因为只有一个 field

### test：
- [[#4794](https://github.com/seata/seata/pull/4794)] 重构代码，尝试修复单元测试 `DataSourceProxyTest.getResourceIdTest()`
Expand Down
2 changes: 1 addition & 1 deletion server/src/main/java/io/seata/server/storage/redis/lock/RedisLocker.java
View file
Open in desktop
Expand Up @@ -359,7 +359,7 @@ private boolean doReleaseLock(String xid, Long branchId) {
Map<String, String> rowKeyMap = jedis.hgetAll(xidLockKey);
rowKeyMap.forEach((branch, rowKey) -> rowKeys.add(rowKey));
} else {
rowKeys.addAll(jedis.hmget(xidLockKey, branchId.toString()));
rowKeys.add(jedis.hget(xidLockKey, branchId.toString()));
}
if (CollectionUtils.isNotEmpty(rowKeys)) {
Pipeline pipelined = jedis.pipelined();
Expand Down