[GitHub] [pinot] gortiz commented on pull request #9012: Add k8s resource request by default

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

gortiz commented on PR #9012:
URL: https://github.com/apache/pinot/pull/9012#issuecomment-1185340366

   I've decided to remove the cpu requests and specify instead `-XX=ActiveProcessorCount=2` in the default jvmOpts.
   
   I think this approach is better because the request approach consumed at least 8 CPUs. This means that, by default, the chart would not start in clusters with less than 8 CPUs in total, which can be very common in local k8s installations (like minikube or microk8s) and also in small aws machines.
   
   Specifically, with this change:
   - Nodes with a lot of cpus will not be affected. Anyway, in this case it is expected that a custom `-XX=ActiveProcessorCount` will be used to determine how many threads each container will use.
   - Nodes with few cpus will have installations that use quite more threads than cpus. This could introduce performance issues due to high context switching when several queries are being executed at the same time, but the system should run well enough to do some tests, which is the target of this kind of installations.


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org