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

关于限流描述的疑点 #5

Open
duansheli opened this issue Jan 1, 2020 · 2 comments
Open

关于限流描述的疑点 #5

duansheli opened this issue Jan 1, 2020 · 2 comments

Comments

@duansheli
Copy link

关于这段话

原因是,我们这里测试的代码使用了多线程,注意看 threadCount 的值,一共有32个线程来模拟,而在RunTask的run方法中执行资源保护时,即在 SphU.entry 的内部是没有加锁的,所以就会导致在高并发下,pass的数量会高于20。

这里说没有锁导致了高并发的限流和期望值不一样


这算是sentinel的bug
还是说 SphU.entry 这个api并不是为多线程场景准备的
属于错误用法 而正确的用法就是在单线程下使用 SphU.entry

@all4you
Copy link
Owner

all4you commented Jan 2, 2020

或许我该说,demo是单独一个线程去统计pass数的,而且是每秒钟统计一次,这个本事就存在一定的误差,要想得到准确的结果,应该看metric中的统计值。

@coderlongren
Copy link

@all4you 也就是说 真正部署sentinel集群, 用metric 滑动窗口统计的时候 , 就没问题了是吧?

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

3 participants