> On Янв. 25, 2016, 5:23 п.п., Sid Wagle wrote: > > Looks good, could also add Jonathan Hurely to reviewers, please? > > It seems the ams alert is still quite close to HDFS vs being generic, what > > are your thoughts about instoducing inheritance here? Can we add the > > generic implementation to the resource_managemenent library ?
I can remove HDFS related code and make the alert generic. Actually +1 from me for making it generic. This required to have ability to run the alter only on active namenode, if it's not the case - we can get rid of all the HDFS related code here. This alert can be implemented no as a script as well. - Dmytro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42706/#review116118 ----------------------------------------------------------- On Янв. 25, 2016, 5:28 п.п., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42706/ > ----------------------------------------------------------- > > (Updated Янв. 25, 2016, 5:28 п.п.) > > > Review request for Ambari, Jonathan Hurley and Sid Wagle. > > > Repository: ambari > > > Description > ------- > > Implement script alert based on AMS metrics standard deviation > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/alerts/alert_metrics_deviation.py > PRE-CREATION > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/alerts.json > 1eda00f > > Diff: https://reviews.apache.org/r/42706/diff/ > > > Testing > ------- > > Unit tests passed > > > Thanks, > > Dmytro Sen > >
