morningman commented on a change in pull request #6443:
URL: https://github.com/apache/incubator-doris/pull/6443#discussion_r695327788



##########
File path: gensrc/thrift/PaloInternalService.thrift
##########
@@ -77,6 +77,10 @@ struct TLoadErrorHubInfo {
     3: optional TBrokerErrorHubInfo broker_info;
 }
 
+struct TResourceLimit {
+    1: optional i32 cpu_limit

Review comment:
       > maybe use max_concurrency is better
   
   In fact, what I want to express is that this value is not bound to any 
implementation. It does not represent the number of threads or the number of 
concurrency, but a relative value.
   It is more like a priority concept, or similar to the concept of weight in 
cgroups.
   In this way, we can continue to use this parameter when we implement more 
refined resource isolation in the future.




-- 
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-unsubscr...@doris.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to