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

zhangliang 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 89b80267c75 Rename 
ExplainStatementContext.explainableSQLStatementContext (#35718)
89b80267c75 is described below

commit 89b80267c756ae50d4fdef6d7c3a2a0ae12b8451
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Sun Jun 15 21:37:57 2025 +0800

    Rename ExplainStatementContext.explainableSQLStatementContext (#35718)
---
 .../shardingsphere/sharding/decider/ShardingSQLFederationDecider.java | 4 ++--
 .../binder/context/statement/type/dal/ExplainStatementContext.java    | 4 ++--
 .../shardingsphere/single/decider/SingleSQLFederationDecider.java     | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/decider/ShardingSQLFederationDecider.java
 
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/decider/ShardingSQLFederationDecider.java
index e97103f029f..160c40320c0 100644
--- 
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/decider/ShardingSQLFederationDecider.java
+++ 
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/decider/ShardingSQLFederationDecider.java
@@ -48,10 +48,10 @@ public final class ShardingSQLFederationDecider implements 
SQLFederationDecider<
             return decide0((SelectStatementContext) sqlStatementContext, 
database, rule, includedDataNodes);
         } else if (sqlStatementContext instanceof ExplainStatementContext) {
             ExplainStatementContext explainStatementContext = 
(ExplainStatementContext) sqlStatementContext;
-            
ShardingSpherePreconditions.checkState(explainStatementContext.getSqlStatementContext()
 instanceof SelectStatementContext,
+            
ShardingSpherePreconditions.checkState(explainStatementContext.getExplainableSQLStatementContext()
 instanceof SelectStatementContext,
                     () -> new UnsupportedSQLOperationException(
                             String.format("unsupported Explain %s statement in 
sql federation", 
explainStatementContext.getSqlStatement().getExplainableSQLStatement())));
-            return decide0((SelectStatementContext) 
explainStatementContext.getSqlStatementContext(), database, rule, 
includedDataNodes);
+            return decide0((SelectStatementContext) 
explainStatementContext.getExplainableSQLStatementContext(), database, rule, 
includedDataNodes);
         }
         throw new UnsupportedSQLOperationException(String.format("unsupported 
SQL statement %s in sql federation", sqlStatementContext.getSqlStatement()));
     }
diff --git 
a/infra/binder/core/src/main/java/org/apache/shardingsphere/infra/binder/context/statement/type/dal/ExplainStatementContext.java
 
b/infra/binder/core/src/main/java/org/apache/shardingsphere/infra/binder/context/statement/type/dal/ExplainStatementContext.java
index 8d9ab6c81d8..fd08d542c9e 100644
--- 
a/infra/binder/core/src/main/java/org/apache/shardingsphere/infra/binder/context/statement/type/dal/ExplainStatementContext.java
+++ 
b/infra/binder/core/src/main/java/org/apache/shardingsphere/infra/binder/context/statement/type/dal/ExplainStatementContext.java
@@ -39,13 +39,13 @@ public final class ExplainStatementContext extends 
CommonSQLStatementContext {
     
     private final TablesContext tablesContext;
     
-    private final SQLStatementContext sqlStatementContext;
+    private final SQLStatementContext explainableSQLStatementContext;
     
     public ExplainStatementContext(final ShardingSphereMetaData metaData, 
final DatabaseType databaseType, final ExplainStatement sqlStatement, final 
List<Object> params,
                                    final String currentDatabaseName) {
         super(databaseType, sqlStatement);
         tablesContext = new 
TablesContext(extractTablesFromExplain(sqlStatement));
-        sqlStatementContext = SQLStatementContextFactory.newInstance(metaData, 
databaseType, sqlStatement.getExplainableSQLStatement(), params, 
currentDatabaseName);
+        explainableSQLStatementContext = 
SQLStatementContextFactory.newInstance(metaData, databaseType, 
sqlStatement.getExplainableSQLStatement(), params, currentDatabaseName);
     }
     
     private Collection<SimpleTableSegment> extractTablesFromExplain(final 
ExplainStatement sqlStatement) {
diff --git 
a/kernel/single/core/src/main/java/org/apache/shardingsphere/single/decider/SingleSQLFederationDecider.java
 
b/kernel/single/core/src/main/java/org/apache/shardingsphere/single/decider/SingleSQLFederationDecider.java
index 6cb658e4e54..914fb9d92c5 100644
--- 
a/kernel/single/core/src/main/java/org/apache/shardingsphere/single/decider/SingleSQLFederationDecider.java
+++ 
b/kernel/single/core/src/main/java/org/apache/shardingsphere/single/decider/SingleSQLFederationDecider.java
@@ -50,10 +50,10 @@ public final class SingleSQLFederationDecider implements 
SQLFederationDecider<Si
             return decide0((SelectStatementContext) sqlStatementContext, 
database, rule, includedDataNodes);
         } else if (sqlStatementContext instanceof ExplainStatementContext) {
             ExplainStatementContext explainStatementContext = 
(ExplainStatementContext) sqlStatementContext;
-            
ShardingSpherePreconditions.checkState(explainStatementContext.getSqlStatementContext()
 instanceof SelectStatementContext,
+            
ShardingSpherePreconditions.checkState(explainStatementContext.getExplainableSQLStatementContext()
 instanceof SelectStatementContext,
                     () -> new UnsupportedSQLOperationException(
                             String.format("unsupported Explain %s statement in 
sql federation", 
explainStatementContext.getSqlStatement().getExplainableSQLStatement())));
-            return decide0((SelectStatementContext) 
explainStatementContext.getSqlStatementContext(), database, rule, 
includedDataNodes);
+            return decide0((SelectStatementContext) 
explainStatementContext.getExplainableSQLStatementContext(), database, rule, 
includedDataNodes);
         }
         throw new UnsupportedSQLOperationException(String.format("unsupported 
SQL statement %s in sql federation", sqlStatementContext.getSqlStatement()));
     }

Reply via email to