[jboss-cvs] JBossAS SVN: r110505 - branches/JBPAPP_5_1/connector/src/main/org/jboss/resource/connectionmanager.

[jboss-cvs-commits at lists.jboss.org]

Author: jesper.pedersen
Date: 2011-02-01 10:26:22 -0500 (Tue, 01 Feb 2011)
New Revision: 110505

Modified:
   branches/JBPAPP_5_1/connector/src/main/org/jboss/resource/connectionmanager/ManagedConnectionFactoryDeployment.java
Log:
[JBPAPP-5292] Verify recovery setup during start

Modified: branches/JBPAPP_5_1/connector/src/main/org/jboss/resource/connectionmanager/ManagedConnectionFactoryDeployment.java
===================================================================
--- branches/JBPAPP_5_1/connector/src/main/org/jboss/resource/connectionmanager/ManagedConnectionFactoryDeployment.java	2011-02-01 11:23:30 UTC (rev 110504)
+++ branches/JBPAPP_5_1/connector/src/main/org/jboss/resource/connectionmanager/ManagedConnectionFactoryDeployment.java	2011-02-01 15:26:22 UTC (rev 110505)
@@ -505,8 +505,16 @@
                }
             }
 
-            getXAResourceRecoveryRegistry().addXAResourceRecovery(this);
-            recoveryRegistered = true;
+            if ((recoverUserName != null && recoverPassword != null) ||
+                (recoverSecurityDomain != null))
+            {
+               getXAResourceRecoveryRegistry().addXAResourceRecovery(this);
+               recoveryRegistered = true;
+            }
+            else
+            {
+               log.warn("Unable to register recovery for " + dmd.getJndiName());
+            }
          }
       }
    }