----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56552/#review165321 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Feb. 10, 2017, 12:41 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56552/ > ----------------------------------------------------------- > > (Updated Feb. 10, 2017, 12:41 p.m.) > > > Review request for Ambari, Nate Cole and Robert Levas. > > > Bugs: AMBARI-19967 > https://issues.apache.org/jira/browse/AMBARI-19967 > > > Repository: ambari > > > Description > ------- > > The `AlertDataManagerTest` is exposing a problem with concurrent creation of > alert groups where an exception is thrown during concurrent UT creation due > to a UNIQUE violation: > > javax.persistence.PersistenceException: > Exception [EclipseLink-4002] (Eclipse Persistence Services - > 2.6.2.v20151217-774c696): org.eclipse.persistence.exceptions.DatabaseException > Internal Exception: org.h2.jdbc.JdbcSQLException: Unique index or primary key > violation: "UNQ_ALERT_GROUP_0_INDEX_9 ON AMBARI.ALERT_GROUP(CLUSTER_ID, > GROUP_NAME) VALUES (1, 'HDFS', 2)"; SQL statement: > INSERT INTO alert_group (group_id, cluster_id, group_name, is_default, > service_name) VALUES (?, ?, ?, ?, ?) [23505-193] > Error Code: 23505 > Call: INSERT INTO alert_group (group_id, cluster_id, group_name, is_default, > service_name) VALUES (?, ?, ?, ?, ?) > bind => [5 parameters bound] > at > org.apache.ambari.server.state.cluster.AlertDataManagerTest.setup(AlertDataManagerTest.java:149) > > This is actually a problem both in the tests and the code for Ambari. We can > replace one of the locks and also synchronize the event publishing. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAO.java > 703ff58 > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDispatchDAO.java > 3b9c97a > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertGroupEntity.java > b660631 > > ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDispatchDAOTest.java > 1ec6d40 > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/AlertDataManagerTest.java > 4ad93e6 > > Diff: https://reviews.apache.org/r/56552/diff/ > > > Testing > ------- > > Results : > > Tests run: 4912, Failures: 0, Errors: 0, Skipped: 39 > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 17:10 min > [INFO] Finished at: 2017-02-10T11:42:49-05:00 > [INFO] Final Memory: 62M/662M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Jonathan Hurley > >
