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.

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

【建议】把WxMpInRedisConfigStorage里面3个lock修改成基于redis的分布式锁 #181

Closed
yiyi910 opened this issue Apr 12, 2017 · 4 comments

Comments

@yiyi910
Copy link

yiyi910 commented Apr 12, 2017

public WxMpInRedisConfigStorage() {
this.accessTokenLock = new ReentrantLock();
this.jsapiTicketLock = new ReentrantLock();
this.cardApiTicketLock = new ReentrantLock();
}

@lly835
Copy link

lly835 commented Apr 14, 2017

看最新的代码,已去掉锁机制,用Jedis直接操作

@xusheng1987
Copy link

没有去掉锁机制啊

@ylmotol7
Copy link

基于redis的锁?不合理吧?能扩展就扩展

@binarywang
Copy link
Owner

暂不考虑,redis的配置本来只是仅供参考,如有需要自行实现即可

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants