[GitHub] [lucene-solr] atris edited a comment on pull request #1686: SOLR-13528: Implement Request Rate Limiters

[GitBox <gi...@apache.org>]

atris edited a comment on pull request #1686:
URL: https://github.com/apache/lucene-solr/pull/1686#issuecomment-668121187


   @madrob I spent some time thinking and decided that we should not be blocking rejected requests but instead returning them immediately with 429 (Too Many Requests) HTTP code. Here is why:
   
   1. User should have the ability to control the behaviour in case of a quota breach.
   2. Adding queuing logic to Solr seems as an anti pattern -- are we over stretching the boundaries of our responsibility?
   3. This might create a false impression that Solr has slowed down since request time will asymptotically increase before timeouts begin to happen. Rejecting upfront will keep the users with reality.
   4. In many scenarios, it is important for the user to know the status of their resource usage so as to adjust accordingly.
   
   Accordingly, I have updated the PR. Apologies for the delay. Please take a look and let me know your thoughts and comments.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org