Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12120 )
Change subject: IMPALA-7941: part 1: detect cgroups memory limit ...................................................................... IMPALA-7941: part 1: detect cgroups memory limit This adds the logic to detect the cgroups memory limit, but does not use it to set the process memory limit yet. The information obtained is logged at startup and accessible through the Web UI. The patch boils down to reading from the appropriate places in the filesystem. The main complication is that paths need to be translated to point to the right cgroup node inside the container. This deletes some useless cgroup logic from ProcessStateInfo that printed whatever happened to be the last cgroup in a file. Testing: Added a unit test to check that the code could parse the cgroups hierarchy ok and return a positive value. Tested on CentOS6 and CentOS7. Change-Id: Ic0f5ed0e94511361bf5605822c0874c16c45ffa9 Reviewed-on: http://gerrit.cloudera.org:8080/12120 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/common/init.cc M be/src/util/CMakeLists.txt A be/src/util/cgroup-util.cc A be/src/util/cgroup-util.h M be/src/util/default-path-handlers.cc M be/src/util/proc-info-test.cc M be/src/util/process-state-info.cc M be/src/util/process-state-info.h M www/root.tmpl 9 files changed, 289 insertions(+), 52 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/12120 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic0f5ed0e94511361bf5605822c0874c16c45ffa9 Gerrit-Change-Number: 12120 Gerrit-PatchSet: 15 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Pooja Nilangekar <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
