248 Star 1.8K Fork 606

GVP京东零售 / hotkey

 / 详情

客户端并发疑问

Backlog
Task
Opened this issue  
2020-12-30 09:46

这个客户端统计次数的代码存在并发问题,是为了性能故意这样设计的吗?

model.setCount(model.getCount() + hotKeyModel.getCount());

Comments (4)

gyf created任务
gyf set related repository to 京东-平台业务中心/hotkey
Expand operation logs

你没仔细看,他不存在并发问题。他是单线程在调用

你没仔细看,他不存在并发问题。他是单线程在调用

@tianyaleixiaowu 没有看到加锁的地方,请教一下,是哪个线程?我看JdHotKeyStore.isHotKey(String key)就会调用这个

我看错了,这个collect不是单线程在调用,lockAndGetResult是单线程调用。

putIfAbsent方法是线程安全的,他在那一刻有个加锁的操作。在非极端情况下,不会出现加错数量的问题。
可以通过写demo试试

Sign in to comment

Status
Assignees
Projects
Milestones
Pull Requests
Successfully merging a pull request will close this issue.
Branches
Planed to start   -   Planed to end
-
Top level
Priority
Duration (hours)
Confirm
参与者(2)
303698 tianyalei 1578919857
Java
1
https://gitee.com/jd-platform-opensource/hotkey.git
git@gitee.com:jd-platform-opensource/hotkey.git
jd-platform-opensource
hotkey
hotkey

Search

181749 a2d7925e 1850385 181749 9f8568a7 1850385