----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23654/#review48052 -----------------------------------------------------------
Looks good... just some nit-picky stuff. ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java <https://reviews.apache.org/r/23654/#comment84311> public method should have doc. ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDispatchDAO.java <https://reviews.apache.org/r/23654/#comment84312> Is the @RequiresSession annotation needed? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertDefinitionEntity.java <https://reviews.apache.org/r/23654/#comment84315> Would isEnabled be a better name? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertDefinitionEntity.java <https://reviews.apache.org/r/23654/#comment84317> Empty doc? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertDefinitionEntity.java <https://reviews.apache.org/r/23654/#comment84318> Empty doc? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertGroupEntity.java <https://reviews.apache.org/r/23654/#comment84322> Empty doc? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertGroupEntity.java <https://reviews.apache.org/r/23654/#comment84324> Empty doc? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertHistoryEntity.java <https://reviews.apache.org/r/23654/#comment84326> Empty doc? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertHistoryEntity.java <https://reviews.apache.org/r/23654/#comment84327> Empty doc? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertNoticeEntity.java <https://reviews.apache.org/r/23654/#comment84329> Empty doc? ... okay I'll stop pointing these out :) ambari-server/src/main/java/org/apache/ambari/server/state/alert/AlertDefinition.java <https://reviews.apache.org/r/23654/#comment84330> Docs? and I don't think we are supposed to use @author. ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql <https://reviews.apache.org/r/23654/#comment84332> How come only the postgres scripts? - Tom Beerbower On July 17, 2014, 7:47 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23654/ > ----------------------------------------------------------- > > (Updated July 17, 2014, 7:47 p.m.) > > > Review request for Ambari, Jonathan Hurley and Tom Beerbower. > > > Bugs: AMBARI-6528 > https://issues.apache.org/jira/browse/AMBARI-6528 > > > Repository: ambari > > > Description > ------- > > Added endpoint and ResourceProvider for alert definitions. This is really > just to get the endpoint going, and all the other CRUD operations will be in > another JIRA. Also updated some DAO, Entity and SQL fixes from Jonathan. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/AlertDefResourceDefinition.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/ClusterResourceDefinition.java > 43578c6 > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java > 2a87c4f > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AlertDefinitionService.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java > 9bbebf5 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterService.java > 29ca8a0 > > ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java > 15c382b > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java > d71cb8c > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java > f68f21c > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertDefinitionResourceProvider.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java > b70f335 > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAO.java > 6f084df > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDispatchDAO.java > c5a9186 > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertsDAO.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertCurrentEntity.java > aa99a08 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertDefinitionEntity.java > 98af71d > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertGroupEntity.java > 6df6be2 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertHistoryEntity.java > c1e346a > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertNoticeEntity.java > bc3958e > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertTargetEntity.java > 595a3b8 > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java > 44bc369 > > ambari-server/src/main/java/org/apache/ambari/server/state/alert/AlertDefinition.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/alert/MetricAlert.java > PRE-CREATION > ambari-server/src/main/java/org/apache/ambari/server/state/alert/Scope.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/alert/SourceType.java > PRE-CREATION > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 5a42153 > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql > deb70ab > ambari-server/src/main/resources/key_properties.json 68aa632 > ambari-server/src/main/resources/properties.json a6cdac4 > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/alerts.json > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/api/resources/ClusterResourceDefinitionTest.java > a4ee74b > > ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java > 71bc1f8 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AlertDefinitionResourceProviderTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAOTest.java > b34a7dd > ambari-server/src/test/resources/stacks/HDP/2.0.5/services/HDFS/alerts.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/23654/diff/ > > > Testing > ------- > > Unit test and manual testing against the endpoint. > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 15:53.464s > [INFO] Finished at: Thu Jul 17 15:34:30 EDT 2014 > [INFO] Final Memory: 21M/123M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Nate Cole > >
