GabrielCWT opened a new pull request, #18613: URL: https://github.com/apache/druid/pull/18613
This PR adds 2 new fields to the sys.servers table which is displayed on the web console. These will be stored on `DiscoveryDruidNode` 1. Available Processors 2. Total Memory (Displayed in **binary** bytes) Both fields represent the available resource on the machine. If the service is hosted in a container, it will represent the resource available to the container instead. (Container Aware) The reason for adding this feature is so that we are able to know the cost of running each of the servers. This is useful when deploying on Kubernetes as we have each server running on an isolated pod. Interestingly, it seems that from Java 17, the `OPERATING_SYSTEM_MX_BEAN` is container aware which helps to simplify the implementation. It might be possible to simplify some legacy code which can be found in the `cgroups` folder i.e. `Cpu.java` and `Memory.java`. I have not looked into detail but I feel that it is something worth looking into. https://developers.redhat.com/articles/2022/04/19/java-17-whats-new-openjdks-container-awareness#recent_changes_in_openjdk_s_container_awareness_code -- 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...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
