Author: agomes
Date: Sun Sep  3 18:40:36 2017
New Revision: 1807159

URL: http://svn.apache.org/viewvc?rev=1807159&view=rev
Log:
Add a list for "JDBC Connection Configuration" "Validation Query" attribut. 
Felix remarks implemented

Modified:
    
jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/config/DataSourceElementBeanInfo.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/component_reference.xml

Modified: 
jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/config/DataSourceElementBeanInfo.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/config/DataSourceElementBeanInfo.java?rev=1807159&r1=1807158&r2=1807159&view=diff
==============================================================================
--- 
jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/config/DataSourceElementBeanInfo.java
 (original)
+++ 
jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/config/DataSourceElementBeanInfo.java
 Sun Sep  3 18:40:36 2017
@@ -90,9 +90,8 @@ public class DataSourceElementBeanInfo e
         p.setValue(DEFAULT, "5000");
         p = property("checkQuery", TypeEditor.ComboStringEditor);
         p.setValue(NOT_UNDEFINED, Boolean.TRUE);
-        p.setValue(DEFAULT, "Select 1");
-        String[] CHECK_QUERY = getCheckQuery();
-        p.setValue(TAGS, CHECK_QUERY);
+        p.setValue(DEFAULT, "");
+        p.setValue(TAGS, 
JOrphanUtils.split(JMeterUtils.getPropDefault("jdbc.config.check.query", ""), 
"|"));
         p = property("dbUrl");
         p.setValue(NOT_UNDEFINED, Boolean.TRUE);
         p.setValue(DEFAULT, "");
@@ -136,13 +135,4 @@ public class DataSourceElementBeanInfo e
         }
         return -1;
     }
-
-    /**
-     * Get the check queris for the main databases
-     * Based in 
https://stackoverflow.com/questions/10684244/dbcp-validationquery-for-different-databases
-     * @return a String[] with the list of check queries
-     */
-    private String[] getCheckQuery() {
-        return 
JOrphanUtils.split(JMeterUtils.getPropDefault("jdbc.config.check.query", ""), 
"|"); //$NON-NLS-1$
-    }
 }

Modified: jmeter/trunk/xdocs/changes.xml
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1807159&r1=1807158&r2=1807159&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
+++ jmeter/trunk/xdocs/changes.xml [utf-8] Sun Sep  3 18:40:36 2017
@@ -103,6 +103,7 @@ Incorporated feed back about unclear doc
 <ul>
     <li><bug>61320</bug>Test Action : Set duration to <code>0</code> by 
default</li>
     <li>JDBC Connection Configuration : Set Max Number of Connections to 
<code>0</code> by default</li>
+    <li>JDBC Connection Configuration : Set "Validation Query" to 
<code>empty</code> by default to use isValid method of JDBC driver</li>
     <li>JDBC Connection Configuration : Add a list for main databases 
validation queries for "Validation Query" attribute</li>
 </ul>
 

Modified: jmeter/trunk/xdocs/usermanual/component_reference.xml
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/xdocs/usermanual/component_reference.xml?rev=1807159&r1=1807158&r2=1807159&view=diff
==============================================================================
--- jmeter/trunk/xdocs/usermanual/component_reference.xml (original)
+++ jmeter/trunk/xdocs/usermanual/component_reference.xml Sun Sep  3 18:40:36 
2017
@@ -4029,19 +4029,19 @@ instead. (see figures 12 and 13).</p>
         This defaults to the '<code>isValid()</code>' method of the jdbc 
driver, which is suitable for many databases.
         However some may require a different query; for example Oracle 
something like '<code>SELECT 1 FROM DUAL</code>' could be used.
         <p>The list of the validation queries can be configured with 
<code>jdbc.config.check.query</code> property and are by default:</p>
-            <ul>
-              <li>hsqldb - select 1 from INFORMATION_SCHEMA.SYSTEM_USERS</li>
-              <li>Oracle - select 1 from dual</li>
-              <li>DB2 - select 1 from sysibm.sysdummy1</li>
-              <li>mysql - select 1</li>
-              <li>microsoft SQL Server - select 1</li>
-              <li>postgresql - select 1</li>
-              <li>ingres - select 1</li>
-              <li>derby - values 1</li>
-              <li>H2 - select 1</li>
-              <li>Firebird - select 1 from rdb$database</li>
-            </ul>
-        <note>The list come from <a 
href="https://stackoverflow.com/questions/10684244/dbcp-validationquery-for-different-databases";>https://stackoverflow.com/questions/10684244/dbcp-validationquery-for-different-databases</a>
 and it can be incorrect</note>
+            <dl>
+              <dt>hsqldb</dt><dd>select 1 from 
INFORMATION_SCHEMA.SYSTEM_USERS</dd>
+              <dt>Oracle</dt><dd>select 1 from dual</dd>
+              <dt>DB2</dt><dd>select 1 from sysibm.sysdummy1</dd>
+              <dt>mysql</dt><dd>select 1</dd>
+              <dt>microsoft SQL Server</dt><dd>select 1</dd>
+              <dt>postgresql</dt><dd>select 1</dd>
+              <dt>ingres</dt><dd>select 1</dd>
+              <dt>derby</dt><dd>values 1</dd>
+              <dt>H2</dt><dd>select 1</dd>
+              <dt>Firebird</dt><dd>select 1 from rdb$database</dd>
+            </dl>
+        <note>The list come from <a 
href="https://stackoverflow.com/questions/10684244/dbcp-validationquery-for-different-databases";>stackoverflow
 entry on different database validation queries</a> and it can be 
incorrect</note>
         <note>Note this validation query is used on pool creation to validate 
it even if "<code>Test While Idle</code>" suggests query would only be used on 
idle connections.
         This is DBCP behaviour.</note>
         </property>


Reply via email to