Hello Pooja Nilangekar, Philip Zeyliger, Bikramjeet Vig, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12120
to look at the new patch set (#12).
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
---
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, 285 insertions(+), 52 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/20/12120/12
--
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: newpatchset
Gerrit-Change-Id: Ic0f5ed0e94511361bf5605822c0874c16c45ffa9
Gerrit-Change-Number: 12120
Gerrit-PatchSet: 12
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Pooja Nilangekar <pooja.nilange...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
