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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
