[ 
https://issues.apache.org/jira/browse/SCB-651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16512056#comment-16512056
 ] 

ASF GitHub Bot commented on SCB-651:
------------------------------------

limbojinne opened a new pull request #770: [SCB-651] Fixes bug of qps rate 
limit value
URL: https://github.com/apache/incubator-servicecomb-java-chassis/pull/770
 
 
   Added the logic for the first cycle of qps controller so that the number of 
requests allowed per second equals to its limit value.
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily:
   
    - [x] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/browse/SCB) filed for the change (usually 
before you start working on it).  Trivial changes like typos do not require a 
JIRA issue.  Your pull request should address just this issue, without pulling 
in other changes.
    - [x] Each commit in the pull request should have a meaningful subject line 
and body.
    - [x] Format the pull request title like `[SCB-XXX] Fixes bug in 
ApproximateQuantiles`, where you replace `SCB-XXX` with the appropriate JIRA 
issue.
    - [x] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
    - [x] Run `mvn clean install` to make sure basic checks pass. A more 
thorough check will be performed on your pull request automatically.
    - [x] If this contribution is large, please file an Apache [Individual 
Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   ---
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> QPS rate limit bug
> ------------------
>
>                 Key: SCB-651
>                 URL: https://issues.apache.org/jira/browse/SCB-651
>             Project: Apache ServiceComb
>          Issue Type: Bug
>          Components: Java-Chassis
>    Affects Versions: java-chassis-1.0.0-m1
>            Reporter: Sean Li
>            Assignee: Sean Li
>            Priority: Major
>         Attachments: limit value as 1.jpeg, limit value as 2.jpeg
>
>
> When setting _servicecomb.flowcontrol.Provider.qps.limit.[ServiceName]_, the 
> actual number of requests allowed per second are 1 less than the limit value. 
>  
> For example, if I set _flowcontrol.Provider.qps.limit._
>  _business-service-consumer_ as 1, I'll get NO successful request at the very 
> beginning, although I would expect 1 successful request instead of none. 
> Similarly, if I set _flowcontrol.Provider.qps.limit._
>  _business-service-consumer_ as 2, I'll get 1 successful request instead of 
> 2. 
>   
>  -However, if I pause before sending requests, for example add 
> _Thread.sleep(100)_ before invoking the first request, the number of requests 
> allowed per second will be correct this time. (i.e. 1 successful request when 
> limit value is 1, 2 successful requests when limit value is 2, etc.)-
>   



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to