(shardingsphere) branch master updated: Optimize DataSourcePoolPropertiesValidator to ensure that storage units can be connected (#35526)

[chengzhang]

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

chengzhang 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 0fc1e1b7717 Optimize DataSourcePoolPropertiesValidator to ensure that 
storage units can be connected (#35526)
0fc1e1b7717 is described below

commit 0fc1e1b7717d7a6dcfdb996fa9f87ca7e99c0ad9
Author: jiangML <1060319...@qq.com>
AuthorDate: Thu May 29 10:48:53 2025 +0800

    Optimize DataSourcePoolPropertiesValidator to ensure that storage units can 
be connected (#35526)
---
 .../pool/props/validator/DataSourcePoolPropertiesValidator.java         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/infra/data-source-pool/core/src/main/java/org/apache/shardingsphere/infra/datasource/pool/props/validator/DataSourcePoolPropertiesValidator.java
 
b/infra/data-source-pool/core/src/main/java/org/apache/shardingsphere/infra/datasource/pool/props/validator/DataSourcePoolPropertiesValidator.java
index ad50c1fbe0e..f1086fd3b83 100644
--- 
a/infra/data-source-pool/core/src/main/java/org/apache/shardingsphere/infra/datasource/pool/props/validator/DataSourcePoolPropertiesValidator.java
+++ 
b/infra/data-source-pool/core/src/main/java/org/apache/shardingsphere/infra/datasource/pool/props/validator/DataSourcePoolPropertiesValidator.java
@@ -77,8 +77,8 @@ public final class DataSourcePoolPropertiesValidator {
         DataSource dataSource = null;
         try {
             dataSource = DataSourcePoolCreator.create(props);
+            checkFailFast(dataSource);
             if (expectedPrivileges.isEmpty() || 
expectedPrivileges.contains(PrivilegeCheckType.NONE)) {
-                checkFailFast(dataSource);
                 return;
             }
             checkPrivileges(dataSource, props, expectedPrivileges);