[GitHub] [incubator-opendal] morristai commented on pull request #2444: feat(layer): add throttle layer

["morristai (via GitHub)" <gi...@apache.org>]

morristai commented on PR #2444:
URL: https://github.com/apache/incubator-opendal/pull/2444#issuecomment-1588455524

   BTW, there are still 2 todo item comments inside this PR:
   - Can we apply a throttle layer over `oio::Read`/`oio::BlockingRead` operations?
   - When the query is valid but the Decider cannot accommodate it, should we lock the limiter and wait for the wait_time, or should we let other small requests go first?
   
   If you believe that they are not essential, then I will remove them.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

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