----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22090/#review47380 -----------------------------------------------------------
We have no test coverage for HostInfo here - Dmitro Lisnichenko On July 2, 2014, 3:58 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22090/ > ----------------------------------------------------------- > > (Updated July 2, 2014, 3:58 p.m.) > > > Review request for Ambari and Dmitro Lisnichenko. > > > Bugs: AMBARI-5973 > https://issues.apache.org/jira/browse/AMBARI-5973 > > > Repository: ambari > > > Description > ------- > > We need to add checks for Transparent Huge Page. With it enabled, we have > pretty horrible performance, and it's enabled by default on RHEL. > > Reference: <https://access.redhat.com/site/documentation/en- > US/Red_Hat_Enterprise_Linux/6/html/Performance_Tuning_Guide/s-memory- > transhuge.html> > > What we need to do pseudo code: > > if (`cat /sys/kernel/mm/redhat_transparent_hugepage/enabled` != 'never'); > then > echo never > /sys/kernel/mm/redhat_transparent_hugepage/enabled > end > > We need to add a check, and then add the 'echo never' to the python script. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/HostInfo.py 940305a > ambari-server/src/main/java/org/apache/ambari/server/agent/AgentEnv.java > eed901c > > ambari-server/src/test/java/org/apache/ambari/server/agent/AgentResourceTest.java > aba2f62 > > Diff: https://reviews.apache.org/r/22090/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
