svn commit: r1453512 - in /incubator/ambari/trunk: ./ ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ ambari-server/src/main/java/org/apache/ambari/server/state/cluster/

[swagle]

Author: swagle
Date: Wed Mar  6 19:44:34 2013
New Revision: 1453512

URL: http://svn.apache.org/r1453512
Log:
AMBARI-1564. TestActionManager is failing on master branch. (swagle)

Modified:
    incubator/ambari/trunk/CHANGES.txt
    
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
    
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/StageEntity.java
    
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java

Modified: incubator/ambari/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1453512&r1=1453511&r2=1453512&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Wed Mar  6 19:44:34 2013
@@ -421,6 +421,8 @@ Trunk (unreleased changes):
 
  BUG FIXES
 
+ AMBARI-1564. TestActionManager is failing on master branch. (swagle)
+
  AMBARI-1561. API should return nagios_alerts as a JSON, not a 
  stringified JSON. (swagle)
 

Modified: 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java?rev=1453512&r1=1453511&r2=1453512&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
 (original)
+++ 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
 Wed Mar  6 19:44:34 2013
@@ -257,7 +257,7 @@ public class HostRoleCommandEntity {
 
   private StageEntity stage;
 
-  @ManyToOne(cascade = {CascadeType.PERSIST, CascadeType.MERGE})
+  @ManyToOne(cascade = {CascadeType.MERGE})
   @JoinColumns({@JoinColumn(name = "request_id", referencedColumnName = 
"request_id", nullable = false), @JoinColumn(name = "stage_id", 
referencedColumnName = "stage_id", nullable = false)})
   public StageEntity getStage() {
     return stage;

Modified: 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/StageEntity.java
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/StageEntity.java?rev=1453512&r1=1453511&r2=1453512&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/StageEntity.java
 (original)
+++ 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/StageEntity.java
 Wed Mar  6 19:44:34 2013
@@ -99,7 +99,7 @@ public class StageEntity {
 
   private ClusterEntity cluster;
 
-  @ManyToOne(cascade = {CascadeType.PERSIST, CascadeType.MERGE})
+  @ManyToOne(cascade = {CascadeType.MERGE})
   @JoinColumn(name = "cluster_id", referencedColumnName = "cluster_id")
   public ClusterEntity getCluster() {
     return cluster;

Modified: 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java?rev=1453512&r1=1453511&r2=1453512&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 (original)
+++ 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 Wed Mar  6 19:44:34 2013
@@ -153,7 +153,6 @@ public class ClusterImpl implements Clus
     if (svcHostsLoaded) return;
     writeLock.lock();
     try {
-      if (svcHostsLoaded) return;
       LOG.info("Loading Service Host Components");
       if (svcHostsLoaded) return;
       if (services != null) {
@@ -695,6 +694,7 @@ public class ClusterImpl implements Clus
   public void delete() throws AmbariException {
     readWriteLock.writeLock().lock();
     try {
+      refresh();
       deleteAllServices();
       removeEntities();
       allConfigs.clear();