Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-09 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48440/#review136804 --- Ship it! Ship It! - Robert Levas On June 8, 2016, 2:51 p.m.,

Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Eugene Chekanskiy
> On June 8, 2016, 7:47 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog230.java, > > lines 347-351 > > > > > > If this is needed in UpgradeCatalog240, why is

Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Eugene Chekanskiy
> On June 8, 2016, 7:47 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog230.java, > > lines 347-351 > > > > > > If this is needed in UpgradeCatalog240, why is

Re: Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48440/#review136677 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

Review Request 48440: Cluster operator and ServiceAdministrator able to enable/disable/edit alert definitions

2016-06-08 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48440/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-17128 https://issu