This is an automated email from the ASF dual-hosted git repository.

menghaoran pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 7aa2702  #6872, remove connection test `SELECT 1` to improve 
performance
     new 572be6a  Merge pull request #7334 from tuohai666/select_1
7aa2702 is described below

commit 7aa2702048c65b2ec78ab4f639c74f00f77e56d4
Author: Zhang Yonglun <[email protected]>
AuthorDate: Tue Sep 8 21:53:30 2020 +0800

    #6872, remove connection test `SELECT 1` to improve performance
---
 .../org/apache/shardingsphere/dbtest/env/datasource/DataSourceUtil.java | 2 --
 .../shardingsphere/dbtest/env/datasource/ProxyDataSourceUtil.java       | 1 -
 2 files changed, 3 deletions(-)

diff --git 
a/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/DataSourceUtil.java
 
b/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/DataSourceUtil.java
index 220db32..06ff42d 100644
--- 
a/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/DataSourceUtil.java
+++ 
b/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/DataSourceUtil.java
@@ -77,7 +77,6 @@ public final class DataSourceUtil {
         result.setUsername(databaseEnvironment.getUsername());
         result.setPassword(databaseEnvironment.getPassword());
         result.setMaxTotal(2);
-        result.setValidationQuery("SELECT 1");
         if ("Oracle".equals(databaseType.getName())) {
             result.setConnectionInitSqls(Collections.singleton("ALTER SESSION 
SET CURRENT_SCHEMA = " + dataSourceName));
         }
@@ -96,7 +95,6 @@ public final class DataSourceUtil {
         result.setPassword(databaseEnvironment.getPassword());
         result.setMaximumPoolSize(2);
         result.setTransactionIsolation("TRANSACTION_READ_COMMITTED");
-        result.setConnectionTestQuery("SELECT 1");
         if ("Oracle".equals(databaseType.getName())) {
             result.setConnectionInitSql("ALTER SESSION SET CURRENT_SCHEMA = " 
+ dataSourceName);
         }
diff --git 
a/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/ProxyDataSourceUtil.java
 
b/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/ProxyDataSourceUtil.java
index 11e5814..3a18db9 100644
--- 
a/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/ProxyDataSourceUtil.java
+++ 
b/shardingsphere-integration-test/shardingsphere-test-suite/src/test/java/org/apache/shardingsphere/dbtest/env/datasource/ProxyDataSourceUtil.java
@@ -65,7 +65,6 @@ public final class ProxyDataSourceUtil {
         result.setPassword("root");
         result.setMaximumPoolSize(2);
         result.setTransactionIsolation("TRANSACTION_READ_COMMITTED");
-        result.setConnectionTestQuery("SELECT 1");
         if ("Oracle".equals(databaseType.getName())) {
             result.setConnectionInitSql("ALTER SESSION SET CURRENT_SCHEMA = " 
+ dataSourceName);
         }

Reply via email to