[GitHub] [incubator-pegasus] Shuo-Jia commented on pull request #782: feat: split read request thread pool

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

Shuo-Jia commented on pull request #782:
URL: https://github.com/apache/incubator-pegasus/pull/782#issuecomment-895685557


   > Imo separation of different types of operations into multiple pools doesn't essentially solve the problem... It only mitigates some cases...
   > 
   > Think of a complex workload that consists of huge single key get, and small span of multiget. How do y suppose to adjust the thread pools? Will you assign a set of threads separately for each operation?
   
   Yeah, right. Actually we have discussed the problem for different case. However, since the benchmark has showed the improvement, at least it doesn't become worse, we decide apply it and verify its online effect.
   
   `scan query`  and `point query` is different type for `exec time` in most case, so seperate them seem to be appropriate in theory.


-- 
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: dev-unsubscribe@pegasus.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org