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

解除不必要的同步,之前的同步会导致严重性能问题。 #384

Merged
merged 1 commit into from
Nov 24, 2017

Conversation

axeon
Copy link

@axeon axeon commented Nov 23, 2017

之前对于executeInternal全部使用了synchronized,导致在大量用户登录的时候,由于http本身的操作时延,造成大量阻塞,导致系统无法使用。
实际上大量的synchronized 都可以去除,特别对于token的操作,建议由独立进程刷新,也可以通过类似guavaCache的方式,确保token随时可用。

@binarywang binarywang merged commit 920a2f7 into binarywang:develop Nov 24, 2017
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

Successfully merging this pull request may close these issues.

2 participants