(shardingsphere) branch master updated: Refactor SQLBindEngine (#31990)

[jianglongtao]

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

jianglongtao 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 3455f79adf2 Refactor SQLBindEngine (#31990)
3455f79adf2 is described below

commit 3455f79adf2835f58031f4983f761e181b6e4271
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Fri Jul 5 22:19:44 2024 +0800

    Refactor SQLBindEngine (#31990)
---
 .../shardingsphere/infra/binder/engine/SQLBindEngine.java    | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/infra/binder/src/main/java/org/apache/shardingsphere/infra/binder/engine/SQLBindEngine.java
 
b/infra/binder/src/main/java/org/apache/shardingsphere/infra/binder/engine/SQLBindEngine.java
index f9c31c2236e..8c3e892f356 100644
--- 
a/infra/binder/src/main/java/org/apache/shardingsphere/infra/binder/engine/SQLBindEngine.java
+++ 
b/infra/binder/src/main/java/org/apache/shardingsphere/infra/binder/engine/SQLBindEngine.java
@@ -60,24 +60,24 @@ public final class SQLBindEngine {
      * @return SQL statement context
      */
     public SQLStatementContext bind(final SQLStatement sqlStatement, final 
List<Object> params) {
-        SQLStatement buoundedSQLStatement = bind(sqlStatement, metaData, 
currentDatabaseName);
+        SQLStatement buoundedSQLStatement = bind(sqlStatement);
         return SQLStatementContextFactory.newInstance(metaData, params, 
buoundedSQLStatement, currentDatabaseName);
     }
     
-    private SQLStatement bind(final SQLStatement statement, final 
ShardingSphereMetaData metaData, final String currentDatabaseName) {
+    private SQLStatement bind(final SQLStatement statement) {
         if (hintValueContext.findHintDataSourceName().isPresent()) {
             return statement;
         }
         if (statement instanceof DMLStatement) {
-            return bindDMLStatement(statement, metaData, currentDatabaseName);
+            return bindDMLStatement(statement);
         }
         if (statement instanceof DDLStatement) {
-            return bindDDLStatement(statement, metaData, currentDatabaseName);
+            return bindDDLStatement(statement);
         }
         return statement;
     }
     
-    private static SQLStatement bindDMLStatement(final SQLStatement statement, 
final ShardingSphereMetaData metaData, final String currentDatabaseName) {
+    private SQLStatement bindDMLStatement(final SQLStatement statement) {
         if (statement instanceof SelectStatement) {
             return new SelectStatementBinder().bind((SelectStatement) 
statement, metaData, currentDatabaseName);
         }
@@ -96,7 +96,7 @@ public final class SQLBindEngine {
         return statement;
     }
     
-    private static SQLStatement bindDDLStatement(final SQLStatement statement, 
final ShardingSphereMetaData metaData, final String currentDatabaseName) {
+    private SQLStatement bindDDLStatement(final SQLStatement statement) {
         if (statement instanceof CursorStatement) {
             return new CursorStatementBinder().bind((CursorStatement) 
statement, metaData, currentDatabaseName);
         }