This is an automated email from the ASF dual-hosted git repository.
panjuan 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 1b8303c9a17 Refactor DistSQLParserFacade (#29632)
1b8303c9a17 is described below
commit 1b8303c9a1709bf9fc39f9b31a59b7b5cf19df65
Author: Liang Zhang <[email protected]>
AuthorDate: Tue Jan 2 22:12:25 2024 +0800
Refactor DistSQLParserFacade (#29632)
---
.../distsql/parser/facade/BroadcastDistSQLParserFacade.java | 5 -----
.../encrypt/distsql/parser/facade/EncryptDistSQLParserFacade.java | 5 -----
.../mask/distsql/parser/facade/MaskDistSQLParserFacade.java | 5 -----
.../parser/facade/ReadwriteSplittingDistSQLParserFacade.java | 5 -----
.../shadow/distsql/parser/facade/ShadowDistSQLParserFacade.java | 5 -----
.../distsql/update/CreateShardingTableRuleStatementUpdaterTest.java | 6 ++----
.../sharding/distsql/parser/facade/ShardingDistSQLParserFacade.java | 5 -----
.../distsql/parser/facade/AuthorityDistSQLParserFacade.java | 5 -----
.../pipeline/cdc/distsql/parser/facade/CDCDistSQLParserFacade.java | 5 -----
.../distsql/parser/facade/MigrationDistSQLParserFacade.java | 5 -----
.../distsql/parser/facade/GlobalClockDistSQLParserFacade.java | 5 -----
.../single/distsql/parser/facade/SingleDistSQLParserFacade.java | 5 -----
.../distsql/parser/facade/SQLFederationDistSQLParserFacade.java | 5 -----
.../parser/distsql/parser/facade/SQLParserDistSQLParserFacade.java | 5 -----
.../distsql/parser/facade/SQLTranslatorDistSQLParserFacade.java | 5 -----
.../traffic/distsql/parser/facade/TrafficDistSQLParserFacade.java | 5 -----
.../distsql/parser/facade/TransactionDistSQLParserFacade.java | 5 -----
...edDistSQLStatementParserEngine.java => DistSQLParserEngine.java} | 4 ++--
.../distsql/parser/engine/api/DistSQLStatementParserEngine.java | 4 ++--
.../distsql/parser/engine/spi/DistSQLParserFacade.java | 6 +-----
20 files changed, 7 insertions(+), 93 deletions(-)
diff --git
a/features/broadcast/distsql/parser/src/main/java/org/apache/shardingsphere/broadcast/distsql/parser/facade/BroadcastDistSQLParserFacade.java
b/features/broadcast/distsql/parser/src/main/java/org/apache/shardingsphere/broadcast/distsql/parser/facade/BroadcastDistSQLParserFacade.java
index 4316e004a7c..0cd12b7f74d 100644
---
a/features/broadcast/distsql/parser/src/main/java/org/apache/shardingsphere/broadcast/distsql/parser/facade/BroadcastDistSQLParserFacade.java
+++
b/features/broadcast/distsql/parser/src/main/java/org/apache/shardingsphere/broadcast/distsql/parser/facade/BroadcastDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class BroadcastDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return BroadcastDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "broadcast";
- }
}
diff --git
a/features/encrypt/distsql/parser/src/main/java/org/apache/shardingsphere/encrypt/distsql/parser/facade/EncryptDistSQLParserFacade.java
b/features/encrypt/distsql/parser/src/main/java/org/apache/shardingsphere/encrypt/distsql/parser/facade/EncryptDistSQLParserFacade.java
index ab98b9054b0..e7e266214de 100644
---
a/features/encrypt/distsql/parser/src/main/java/org/apache/shardingsphere/encrypt/distsql/parser/facade/EncryptDistSQLParserFacade.java
+++
b/features/encrypt/distsql/parser/src/main/java/org/apache/shardingsphere/encrypt/distsql/parser/facade/EncryptDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class EncryptDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return EncryptDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "encrypt";
- }
}
diff --git
a/features/mask/distsql/parser/src/main/java/org/apache/shardingsphere/mask/distsql/parser/facade/MaskDistSQLParserFacade.java
b/features/mask/distsql/parser/src/main/java/org/apache/shardingsphere/mask/distsql/parser/facade/MaskDistSQLParserFacade.java
index ec366dd944a..af025b348fd 100644
---
a/features/mask/distsql/parser/src/main/java/org/apache/shardingsphere/mask/distsql/parser/facade/MaskDistSQLParserFacade.java
+++
b/features/mask/distsql/parser/src/main/java/org/apache/shardingsphere/mask/distsql/parser/facade/MaskDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class MaskDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return MaskDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "mask";
- }
}
diff --git
a/features/readwrite-splitting/distsql/parser/src/main/java/org/apache/shardingsphere/readwritesplitting/distsql/parser/facade/ReadwriteSplittingDistSQLParserFacade.java
b/features/readwrite-splitting/distsql/parser/src/main/java/org/apache/shardingsphere/readwritesplitting/distsql/parser/facade/ReadwriteSplittingDistSQLParserFacade.java
index cdfa2d47bc2..580339c3740 100644
---
a/features/readwrite-splitting/distsql/parser/src/main/java/org/apache/shardingsphere/readwritesplitting/distsql/parser/facade/ReadwriteSplittingDistSQLParserFacade.java
+++
b/features/readwrite-splitting/distsql/parser/src/main/java/org/apache/shardingsphere/readwritesplitting/distsql/parser/facade/ReadwriteSplittingDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class ReadwriteSplittingDistSQLParserFacade
implements DistSQLParse
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return ReadwriteSplittingDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "readwrite-splitting";
- }
}
diff --git
a/features/shadow/distsql/parser/src/main/java/org/apache/shardingsphere/shadow/distsql/parser/facade/ShadowDistSQLParserFacade.java
b/features/shadow/distsql/parser/src/main/java/org/apache/shardingsphere/shadow/distsql/parser/facade/ShadowDistSQLParserFacade.java
index 969511f5f13..f95221712a5 100644
---
a/features/shadow/distsql/parser/src/main/java/org/apache/shardingsphere/shadow/distsql/parser/facade/ShadowDistSQLParserFacade.java
+++
b/features/shadow/distsql/parser/src/main/java/org/apache/shardingsphere/shadow/distsql/parser/facade/ShadowDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class ShadowDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return ShadowDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "shadow";
- }
}
diff --git
a/features/sharding/distsql/handler/src/test/java/org/apache/shardingsphere/sharding/distsql/update/CreateShardingTableRuleStatementUpdaterTest.java
b/features/sharding/distsql/handler/src/test/java/org/apache/shardingsphere/sharding/distsql/update/CreateShardingTableRuleStatementUpdaterTest.java
index e91abf35d34..c4ec40b2cb2 100644
---
a/features/sharding/distsql/handler/src/test/java/org/apache/shardingsphere/sharding/distsql/update/CreateShardingTableRuleStatementUpdaterTest.java
+++
b/features/sharding/distsql/handler/src/test/java/org/apache/shardingsphere/sharding/distsql/update/CreateShardingTableRuleStatementUpdaterTest.java
@@ -18,17 +18,15 @@
package org.apache.shardingsphere.sharding.distsql.update;
import lombok.SneakyThrows;
-import
org.apache.shardingsphere.infra.exception.core.external.sql.type.kernel.category.DistSQLException;
-import org.apache.shardingsphere.distsql.parser.engine.spi.DistSQLParserFacade;
import org.apache.shardingsphere.distsql.segment.AlgorithmSegment;
import org.apache.shardingsphere.distsql.statement.DistSQLStatement;
import org.apache.shardingsphere.infra.config.algorithm.AlgorithmConfiguration;
import org.apache.shardingsphere.infra.config.rule.RuleConfiguration;
+import
org.apache.shardingsphere.infra.exception.core.external.sql.type.kernel.category.DistSQLException;
import
org.apache.shardingsphere.infra.metadata.database.ShardingSphereDatabase;
import
org.apache.shardingsphere.infra.metadata.database.resource.ResourceMetaData;
import org.apache.shardingsphere.infra.metadata.database.rule.RuleMetaData;
import
org.apache.shardingsphere.infra.rule.identifier.type.DataSourceContainedRule;
-import org.apache.shardingsphere.infra.spi.type.typed.TypedSPILoader;
import org.apache.shardingsphere.sharding.api.config.ShardingRuleConfiguration;
import
org.apache.shardingsphere.sharding.api.config.rule.ShardingAutoTableRuleConfiguration;
import
org.apache.shardingsphere.sharding.api.config.rule.ShardingTableRuleConfiguration;
@@ -290,7 +288,7 @@ class CreateShardingTableRuleStatementUpdaterTest {
private DistSQLStatement getDistSQLStatement(final String sql) {
ShardingDistSQLParserFacade facade = new ShardingDistSQLParserFacade();
ParseASTNode parseASTNode = (ParseASTNode)
SQLParserFactory.newInstance(sql, facade.getLexerClass(),
facade.getParserClass()).parse();
- SQLVisitor visitor =
TypedSPILoader.getService(DistSQLParserFacade.class,
facade.getType()).getVisitorClass().getDeclaredConstructor().newInstance();
+ SQLVisitor visitor =
facade.getVisitorClass().getDeclaredConstructor().newInstance();
return (DistSQLStatement) visitor.visit(parseASTNode.getRootNode());
}
diff --git
a/features/sharding/distsql/parser/src/main/java/org/apache/shardingsphere/sharding/distsql/parser/facade/ShardingDistSQLParserFacade.java
b/features/sharding/distsql/parser/src/main/java/org/apache/shardingsphere/sharding/distsql/parser/facade/ShardingDistSQLParserFacade.java
index 242eb87bbff..8777b3dfcfa 100644
---
a/features/sharding/distsql/parser/src/main/java/org/apache/shardingsphere/sharding/distsql/parser/facade/ShardingDistSQLParserFacade.java
+++
b/features/sharding/distsql/parser/src/main/java/org/apache/shardingsphere/sharding/distsql/parser/facade/ShardingDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class ShardingDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return ShardingDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "sharding";
- }
}
diff --git
a/kernel/authority/distsql/parser/src/main/java/org/apache/shardingsphere/authority/distsql/parser/facade/AuthorityDistSQLParserFacade.java
b/kernel/authority/distsql/parser/src/main/java/org/apache/shardingsphere/authority/distsql/parser/facade/AuthorityDistSQLParserFacade.java
index e348627f1d7..c085d6cc991 100644
---
a/kernel/authority/distsql/parser/src/main/java/org/apache/shardingsphere/authority/distsql/parser/facade/AuthorityDistSQLParserFacade.java
+++
b/kernel/authority/distsql/parser/src/main/java/org/apache/shardingsphere/authority/distsql/parser/facade/AuthorityDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class AuthorityDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return AuthorityDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "authority";
- }
}
diff --git
a/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/cdc/distsql/parser/facade/CDCDistSQLParserFacade.java
b/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/cdc/distsql/parser/facade/CDCDistSQLParserFacade.java
index 1a4681e5964..c8216720ea5 100644
---
a/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/cdc/distsql/parser/facade/CDCDistSQLParserFacade.java
+++
b/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/cdc/distsql/parser/facade/CDCDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class CDCDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return CDCDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "CDC";
- }
}
diff --git
a/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/migration/distsql/parser/facade/MigrationDistSQLParserFacade.java
b/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/migration/distsql/parser/facade/MigrationDistSQLParserFacade.java
index 4269c12869c..93f3a2ed3bb 100644
---
a/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/migration/distsql/parser/facade/MigrationDistSQLParserFacade.java
+++
b/kernel/data-pipeline/distsql/parser/src/main/java/org/apache/shardingsphere/data/pipeline/migration/distsql/parser/facade/MigrationDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class MigrationDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return MigrationDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "migration";
- }
}
diff --git
a/kernel/global-clock/distsql/parser/src/main/java/org/apache/shardingsphere/globalclock/distsql/parser/facade/GlobalClockDistSQLParserFacade.java
b/kernel/global-clock/distsql/parser/src/main/java/org/apache/shardingsphere/globalclock/distsql/parser/facade/GlobalClockDistSQLParserFacade.java
index 294dc0846c1..196b3d8d13f 100644
---
a/kernel/global-clock/distsql/parser/src/main/java/org/apache/shardingsphere/globalclock/distsql/parser/facade/GlobalClockDistSQLParserFacade.java
+++
b/kernel/global-clock/distsql/parser/src/main/java/org/apache/shardingsphere/globalclock/distsql/parser/facade/GlobalClockDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class GlobalClockDistSQLParserFacade implements
DistSQLParserFacade
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return GlobalClockDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "global_clock";
- }
}
diff --git
a/kernel/single/distsql/parser/src/main/java/org/apache/shardingsphere/single/distsql/parser/facade/SingleDistSQLParserFacade.java
b/kernel/single/distsql/parser/src/main/java/org/apache/shardingsphere/single/distsql/parser/facade/SingleDistSQLParserFacade.java
index 81d9c522008..c572f014992 100644
---
a/kernel/single/distsql/parser/src/main/java/org/apache/shardingsphere/single/distsql/parser/facade/SingleDistSQLParserFacade.java
+++
b/kernel/single/distsql/parser/src/main/java/org/apache/shardingsphere/single/distsql/parser/facade/SingleDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class SingleDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return SingleDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "single";
- }
}
diff --git
a/kernel/sql-federation/distsql/parser/src/main/java/org/apache/shardingsphere/sqlfederation/distsql/parser/facade/SQLFederationDistSQLParserFacade.java
b/kernel/sql-federation/distsql/parser/src/main/java/org/apache/shardingsphere/sqlfederation/distsql/parser/facade/SQLFederationDistSQLParserFacade.java
index 61ebda33e05..e9a51e207b4 100644
---
a/kernel/sql-federation/distsql/parser/src/main/java/org/apache/shardingsphere/sqlfederation/distsql/parser/facade/SQLFederationDistSQLParserFacade.java
+++
b/kernel/sql-federation/distsql/parser/src/main/java/org/apache/shardingsphere/sqlfederation/distsql/parser/facade/SQLFederationDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class SQLFederationDistSQLParserFacade
implements DistSQLParserFaca
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return SQLFederationDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "sql_federation";
- }
}
diff --git
a/kernel/sql-parser/distsql/parser/src/main/java/org/apache/shardingsphere/parser/distsql/parser/facade/SQLParserDistSQLParserFacade.java
b/kernel/sql-parser/distsql/parser/src/main/java/org/apache/shardingsphere/parser/distsql/parser/facade/SQLParserDistSQLParserFacade.java
index f1f85f2f264..813340263ae 100644
---
a/kernel/sql-parser/distsql/parser/src/main/java/org/apache/shardingsphere/parser/distsql/parser/facade/SQLParserDistSQLParserFacade.java
+++
b/kernel/sql-parser/distsql/parser/src/main/java/org/apache/shardingsphere/parser/distsql/parser/facade/SQLParserDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class SQLParserDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return SQLParserDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "sql_parser";
- }
}
diff --git
a/kernel/sql-translator/distsql/parser/src/main/java/org/apache/shardingsphere/sqltranslator/distsql/parser/facade/SQLTranslatorDistSQLParserFacade.java
b/kernel/sql-translator/distsql/parser/src/main/java/org/apache/shardingsphere/sqltranslator/distsql/parser/facade/SQLTranslatorDistSQLParserFacade.java
index 0e3529d2d03..4f876d394b4 100644
---
a/kernel/sql-translator/distsql/parser/src/main/java/org/apache/shardingsphere/sqltranslator/distsql/parser/facade/SQLTranslatorDistSQLParserFacade.java
+++
b/kernel/sql-translator/distsql/parser/src/main/java/org/apache/shardingsphere/sqltranslator/distsql/parser/facade/SQLTranslatorDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class SQLTranslatorDistSQLParserFacade
implements DistSQLParserFaca
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return SQLTranslatorDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "sql_translator";
- }
}
diff --git
a/kernel/traffic/distsql/parser/src/main/java/org/apache/shardingsphere/traffic/distsql/parser/facade/TrafficDistSQLParserFacade.java
b/kernel/traffic/distsql/parser/src/main/java/org/apache/shardingsphere/traffic/distsql/parser/facade/TrafficDistSQLParserFacade.java
index 833bab02f4c..986a6a314c3 100644
---
a/kernel/traffic/distsql/parser/src/main/java/org/apache/shardingsphere/traffic/distsql/parser/facade/TrafficDistSQLParserFacade.java
+++
b/kernel/traffic/distsql/parser/src/main/java/org/apache/shardingsphere/traffic/distsql/parser/facade/TrafficDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class TrafficDistSQLParserFacade implements
DistSQLParserFacade {
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return TrafficDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "traffic";
- }
}
diff --git
a/kernel/transaction/distsql/parser/src/main/java/org/apache/shardingsphere/transaction/distsql/parser/facade/TransactionDistSQLParserFacade.java
b/kernel/transaction/distsql/parser/src/main/java/org/apache/shardingsphere/transaction/distsql/parser/facade/TransactionDistSQLParserFacade.java
index fd33a7b673d..29ff6a7ad1d 100644
---
a/kernel/transaction/distsql/parser/src/main/java/org/apache/shardingsphere/transaction/distsql/parser/facade/TransactionDistSQLParserFacade.java
+++
b/kernel/transaction/distsql/parser/src/main/java/org/apache/shardingsphere/transaction/distsql/parser/facade/TransactionDistSQLParserFacade.java
@@ -45,9 +45,4 @@ public final class TransactionDistSQLParserFacade implements
DistSQLParserFacade
public Class<? extends SQLVisitor<ASTNode>> getVisitorClass() {
return TransactionDistSQLStatementVisitor.class;
}
-
- @Override
- public String getType() {
- return "transaction";
- }
}
diff --git
a/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/core/featured/FeaturedDistSQLStatementParserEngine.java
b/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/core/featured/DistSQLParserEngine.java
similarity index 96%
rename from
parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/core/featured/FeaturedDistSQLStatementParserEngine.java
rename to
parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/core/featured/DistSQLParserEngine.java
index 9955ca01d1b..4c5bab610f9 100644
---
a/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/core/featured/FeaturedDistSQLStatementParserEngine.java
+++
b/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/core/featured/DistSQLParserEngine.java
@@ -29,9 +29,9 @@ import
org.apache.shardingsphere.sql.parser.exception.SQLParsingException;
import org.apache.shardingsphere.sql.parser.sql.common.statement.SQLStatement;
/**
- * Featured DistSQL statement parser engine.
+ * DistSQL parser engine.
*/
-public final class FeaturedDistSQLStatementParserEngine {
+public final class DistSQLParserEngine {
/**
* Parse SQL.
diff --git
a/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/api/DistSQLStatementParserEngine.java
b/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/api/DistSQLStatementParserEngine.java
index 3bd13c030aa..3f15586aec9 100644
---
a/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/api/DistSQLStatementParserEngine.java
+++
b/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/api/DistSQLStatementParserEngine.java
@@ -20,7 +20,7 @@ package org.apache.shardingsphere.distsql.parser.engine.api;
import org.antlr.v4.runtime.misc.ParseCancellationException;
import
org.apache.shardingsphere.distsql.parser.core.utility.UtilityDistSQLStatementParserEngine;
import
org.apache.shardingsphere.distsql.parser.core.kernel.KernelDistSQLStatementParserEngine;
-import
org.apache.shardingsphere.distsql.parser.core.featured.FeaturedDistSQLStatementParserEngine;
+import
org.apache.shardingsphere.distsql.parser.core.featured.DistSQLParserEngine;
import org.apache.shardingsphere.sql.parser.exception.SQLParsingException;
import org.apache.shardingsphere.sql.parser.sql.common.statement.SQLStatement;
@@ -40,7 +40,7 @@ public final class DistSQLStatementParserEngine {
return new KernelDistSQLStatementParserEngine().parse(sql);
} catch (final ParseCancellationException | SQLParsingException
ignored) {
try {
- return new FeaturedDistSQLStatementParserEngine().parse(sql);
+ return new DistSQLParserEngine().parse(sql);
} catch (final SQLParsingException ignoredToo) {
return new UtilityDistSQLStatementParserEngine().parse(sql);
}
diff --git
a/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/spi/DistSQLParserFacade.java
b/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/spi/DistSQLParserFacade.java
index 13026a20df3..a142b02c4e6 100644
---
a/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/spi/DistSQLParserFacade.java
+++
b/parser/distsql/engine/src/main/java/org/apache/shardingsphere/distsql/parser/engine/spi/DistSQLParserFacade.java
@@ -18,7 +18,6 @@
package org.apache.shardingsphere.distsql.parser.engine.spi;
import org.apache.shardingsphere.infra.spi.annotation.SingletonSPI;
-import org.apache.shardingsphere.infra.spi.type.typed.TypedSPI;
import org.apache.shardingsphere.sql.parser.api.ASTNode;
import org.apache.shardingsphere.sql.parser.api.parser.SQLLexer;
import org.apache.shardingsphere.sql.parser.api.parser.SQLParser;
@@ -28,7 +27,7 @@ import
org.apache.shardingsphere.sql.parser.api.visitor.SQLVisitor;
* DistSQL parser facade.
*/
@SingletonSPI
-public interface DistSQLParserFacade extends TypedSPI {
+public interface DistSQLParserFacade {
/**
* Get SQL lexer class type.
@@ -50,7 +49,4 @@ public interface DistSQLParserFacade extends TypedSPI {
* @return visitor class
*/
Class<? extends SQLVisitor<ASTNode>> getVisitorClass();
-
- @Override
- String getType();
}
