Use a safe default of the useEquals, works on any VM.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@835974 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/modules/jdbc-pool/doc/jdbc-pool.xml b/modules/jdbc-pool/doc/jdbc-pool.xml
index 8c76a9a..c692de9 100644 (file)
--- a/modules/jdbc-pool/doc/jdbc-pool.xml
+++ b/modules/jdbc-pool/doc/jdbc-pool.xml
@@ -383,9 +383,9 @@
     </attribute>
 
     <attribute name="useEquals" required="false">
-      <p>(boolean) Set to true if you wish the <code>ProxyConnection</code> class to use <code>String.equals</code> instead of 
-         <code>==</code> when comparing method names. This property does not apply to added interceptors as those are configured individually.
-         The default value is <code>false</code>.
+      <p>(boolean) Set to true if you wish the <code>ProxyConnection</code> class to use <code>String.equals</code> and set to <code>false</code>
+         when you wish to use <code>==</code> when comparing method names. This property does not apply to added interceptors as those are configured individually.
+         The default value is <code>true</code>.
       </p> 
     </attribute>    
   </attributes>  
@@ -426,9 +426,9 @@
     <p>Abstract base class for all interceptors, can not be instantiated.</p>
     <attributes>
       <attribute name="useEquals" required="false">
-        <p>(String as boolean) Set to true if you wish the interceptor class to use <code>String.equals</code> instead of 
-         <code>==</code> when comparing method names. This property does not apply to added interceptors as those are configured individually.
-         The default value is <code>false</code>.
+        <p>(boolean) Set to true if you wish the <code>ProxyConnection</code> class to use <code>String.equals</code> and set to <code>false</code>
+         when you wish to use <code>==</code> when comparing method names.
+         The default value is <code>true</code>.
         </p>
       </attribute>
     </attributes>  

diff --git a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/JdbcInterceptor.java b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/JdbcInterceptor.java
index 78a214a..71311bf 100644 (file)
--- a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/JdbcInterceptor.java
+++ b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/JdbcInterceptor.java
@@ -65,7 +65,7 @@ public abstract class JdbcInterceptor implements InvocationHandler {
     /**
      * Property that decides how we do string comparison, default is reference (==)
      */
-    private boolean useEquals = false;
+    private boolean useEquals = true;
 
     /**
      * Public constructor for instantation through reflection

diff --git a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
index 6fad3a3..0f56617 100644 (file)
--- a/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
+++ b/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
@@ -69,7 +69,7 @@ public class PoolProperties implements PoolConfiguration {
     protected boolean testOnConnect =false;
     protected String jdbcInterceptors=null;
     protected boolean fairQueue = true;
-    protected boolean useEquals = false;
+    protected boolean useEquals = true;
     protected int abandonWhenPercentageFull = 0;
     protected long maxAge = 0;
     protected boolean useLock = false;