关于ReferenceResource的问题#hold
protected final AtomicLong refCount = new AtomicLong(1); public synchronized boolean hold() { if (this.isAvailable()) { if (this.refCount.getAndIncrement() > 0) { return true; } else { this.refCount.getAndDecrement(); } }
return false;
}
refCount已经是AtomicLong类型，是否有必要将hold方法放在同步块中
原提问者GitHub用户rushsky518
版权声明：本文内容由阿里云实名注册用户自发贡献，版权归原作者所有，阿里云开发者社区不拥有其著作权，亦不承担相应法律责任。具体规则请查看《阿里云开发者社区用户服务协议》和《阿里云开发者社区知识产权保护指引》。如果您发现本社区中有涉嫌抄袭的内容，填写侵权投诉表单进行举报，一经查实，本社区将立刻删除涉嫌侵权内容。
-
IMO，有必要添加同步，保持方法必须与下面的同步块互斥。否则，hold可能会返回true并清除成功。
rocketmq/store/src/main/java/org/apache/rocketmq/store/ReferenceResource.java
Lines 56 to 65 in 73bb4b4
public void release() { long value = this.refCount.decrementAndGet(); if (value > 0) return;
synchronized (this) { this.cleanupOver = this.cleanup(value); }}
原回答者GitHub用户RongtongJin
2023-05-26 17:42:50赞同 展开评论
阿里云拥有国内全面的云原生产品技术以及大规模的云原生应用实践，通过全面容器化、核心技术互联网化、应用 Serverless 化三大范式，助力制造业企业高效上云，实现系统稳定、应用敏捷智能。拥抱云原生，让创新无处不在。