You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pp...@apache.org on 2011/12/27 18:12:05 UTC
svn commit: r1224948 -
/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProductDerivation.java
Author: ppoddar
Date: Tue Dec 27 17:12:04 2011
New Revision: 1224948
URL: http://svn.apache.org/viewvc?rev=1224948&view=rev
Log:
Wrong logic in setting default remote commit provider
Modified:
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProductDerivation.java
Modified: openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProductDerivation.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProductDerivation.java?rev=1224948&r1=1224947&r2=1224948&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProductDerivation.java (original)
+++ openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProductDerivation.java Tue Dec 27 17:12:04 2011
@@ -702,9 +702,9 @@ public class PersistenceProductDerivatio
String dc = oconf.getDataCache();
String rcp = oconf.getRemoteCommitProvider();
// If the datacache is set and is something other than false
- if (dc != null && dc.equals("false") == false) {
+ if (dc != null && !"false".equals(dc)) {
// If RCP is null or empty, set it to sjvm.
- if (rcp == null || (rcp != null && rcp.equals("") == false)) {
+ if (rcp == null || "".equals(rcp)) {
oconf.setRemoteCommitProvider("sjvm");
}
}