Re: Review Request 48436: Ambari Stale Alert Triggers For Server-Side Performance Alert

2016-06-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48436/ --- (Updated June 9, 2016, 10:07 a.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 48436: Ambari Stale Alert Triggers For Server-Side Performance Alert

2016-06-09 Thread Jonathan Hurley
> On June 9, 2016, 8:47 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/alerts/StaleAlertRunnable.java, > > lines 76-81 > > > > > > Should this be a parameter of the alert definition?

Re: Review Request 48436: Ambari Stale Alert Triggers For Server-Side Performance Alert

2016-06-09 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48436/#review136800 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a