> On July 21, 2014, 2:58 p.m., Tom Beerbower wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/Alert.java, > > lines 55-56 > > <https://reviews.apache.org/r/23690/diff/1/?file=635662#file635662line55> > > > > Is this required? If not can you remove it? > > Dmytro Shkvyra wrote: > Yes Tom, we should have default constructor for deserialization. >
I guess I'm asking if there is a reason that you have to explicitly specify it? Wouldn't the implicit default constructor be the same? - Tom ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23690/#review48227 ----------------------------------------------------------- On July 18, 2014, 6:03 p.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23690/ > ----------------------------------------------------------- > > (Updated July 18, 2014, 6:03 p.m.) > > > Review request for Ambari, Tom Beerbower and Vitalyi Brodetskyi. > > > Bugs: AMBARI-6534 > https://issues.apache.org/jira/browse/AMBARI-6534 > > > Repository: ambari > > > Description > ------- > > During runtime operations, log files and other external factors can > contribute to filling disks that the Hadoop daemons depend on (eg /, > /var/log, /tmp, any yarn-local-dirs). Ambari should watch for all these > locations and warn when they fill up beyond a threshold like 80% > (configurable). > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/Heartbeat.py 65a7a33 > ambari-agent/src/main/python/ambari_agent/HostInfo.py 69e3cae > ambari-agent/src/test/python/ambari_agent/TestHeartbeat.py 4703af4 > ambari-agent/src/test/python/ambari_agent/TestHostInfo.py ba827ae > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java > cdaf947 > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java > 8b8b252 > ambari-server/src/main/java/org/apache/ambari/server/agent/HostStatus.java > d609ecd > > ambari-server/src/main/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProvider.java > 7c40ed1 > ambari-server/src/main/java/org/apache/ambari/server/state/Alert.java > be31893 > > ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java > 752a492 > > ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatMonitor.java > da4a099 > > ambari-server/src/test/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProviderTest.java > cc45ef0 > > Diff: https://reviews.apache.org/r/23690/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Ambari Views ...................................... SUCCESS [7.032s] > [INFO] Ambari Server ..................................... SUCCESS > [18:15.609s] > [INFO] Ambari Agent ...................................... SUCCESS [4.772s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 18:28.464s > [INFO] Finished at: Fri Jul 18 19:11:45 FET 2014 > [INFO] Final Memory: 22M/243M > [INFO] > ------------------------------------------------------------------------ > > > > Thanks, > > Dmitro Lisnichenko > >
