-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31524/
-----------------------------------------------------------

(Updated Feb. 26, 2015, 10:12 p.m.)


Review request for Ambari, Nate Cole and Tom Beerbower.


Bugs: AMBARI-9823
    https://issues.apache.org/jira/browse/AMBARI-9823


Repository: ambari


Description
-------

There seems to be an issue with EclipseLink and nested entities where the 
relationships are not refreshed from the database correctly. This causes a 
problem sending out alert notices for alert groups because the target lookup 
doesn't return the correct entities.

Restarting Ambari Server fixes the issue.


Diffs
-----

  ambari-project/pom.xml 92e3b5c 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAO.java
 2daf804 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertNoticeEntity.java
 d0d4c9a 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertTargetEntity.java
 e1248db 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAOTest.java
 a1cd063 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDispatchDAOTest.java
 44c40f2 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java 
fb3303c 

Diff: https://reviews.apache.org/r/31524/diff/


Testing (updated)
-------

Clean install of a 3-node cluster; verified that history and notices come back 
correctly. Added and removed alert targets with associated notices and verified 
the dispatcher picked up the targets correctly.

mvn clean test

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 14:53 min
[INFO] Finished at: 2015-02-26T21:47:26-05:00
[INFO] Final Memory: 34M/378M
[INFO] ------------------------------------------------------------------------


Thanks,

Jonathan Hurley

Reply via email to