This is an automated email from the ASF dual-hosted git repository.
sunnianjun 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 c0acc6af510 Remove ShardingSphereRule.getType (#28166)
c0acc6af510 is described below
commit c0acc6af510cf01a7bec067b173c84f06035f060
Author: Liang Zhang <[email protected]>
AuthorDate: Fri Aug 18 19:53:14 2023 +0800
Remove ShardingSphereRule.getType (#28166)
---
.../org/apache/shardingsphere/broadcast/rule/BroadcastRule.java | 5 -----
.../java/org/apache/shardingsphere/encrypt/rule/EncryptRule.java | 5 -----
.../main/java/org/apache/shardingsphere/mask/rule/MaskRule.java | 5 -----
.../readwritesplitting/rule/ReadwriteSplittingRule.java | 5 -----
.../java/org/apache/shardingsphere/shadow/rule/ShadowRule.java | 5 -----
.../java/org/apache/shardingsphere/sharding/rule/ShardingRule.java | 5 -----
.../update/CreateShardingTableRuleStatementUpdaterTest.java | 5 -----
.../org/apache/shardingsphere/infra/rule/ShardingSphereRule.java | 7 -------
.../java/org/apache/shardingsphere/infra/fixture/FixtureRule.java | 5 -----
.../infra/metadata/database/rule/ShardingSphereRuleFixture.java | 5 -----
.../database/schema/fixture/rule/TableContainedFixtureRule.java | 5 -----
.../infra/rule/builder/fixture/FixtureDatabaseRule.java | 5 -----
.../infra/rule/builder/fixture/FixtureGlobalRule.java | 5 -----
.../infra/merge/fixture/rule/DecoratorRuleFixture.java | 5 -----
.../shardingsphere/infra/merge/fixture/rule/MergerRuleFixture.java | 5 -----
.../infra/route/fixture/rule/RouteFailureRuleFixture.java | 5 -----
.../shardingsphere/infra/route/fixture/rule/RouteRuleFixture.java | 5 -----
.../org/apache/shardingsphere/authority/rule/AuthorityRule.java | 5 -----
.../shardingsphere/globalclock/core/rule/GlobalClockRule.java | 5 -----
.../java/org/apache/shardingsphere/logging/rule/LoggingRule.java | 5 -----
.../org/apache/shardingsphere/logging/rule/LoggingRuleTest.java | 5 -----
.../java/org/apache/shardingsphere/single/rule/SingleRule.java | 5 -----
.../shardingsphere/sqlfederation/rule/SQLFederationRule.java | 5 -----
.../engine/fixture/rule/SQLFederationDeciderRuleMatchFixture.java | 5 -----
.../fixture/rule/SQLFederationDeciderRuleNotMatchFixture.java | 5 -----
.../java/org/apache/shardingsphere/parser/rule/SQLParserRule.java | 5 -----
.../org/apache/shardingsphere/parser/rule/SQLParserRuleTest.java | 5 -----
.../shardingsphere/sqltranslator/rule/SQLTranslatorRule.java | 5 -----
.../shardingsphere/sqltranslator/rule/SQLTranslatorRuleTest.java | 5 -----
.../shardingsphere/timeservice/core/rule/TimestampServiceRule.java | 5 -----
.../java/org/apache/shardingsphere/traffic/rule/TrafficRule.java | 5 -----
.../apache/shardingsphere/transaction/rule/TransactionRule.java | 5 -----
.../apache/shardingsphere/mode/fixture/FixtureDatabaseRule.java | 5 -----
.../rdl/storage/unit/UnregisterStorageUnitBackendHandlerTest.java | 4 ----
.../apache/shardingsphere/test/fixture/infra/rule/MockedRule.java | 5 -----
35 files changed, 176 deletions(-)
diff --git
a/features/broadcast/core/src/main/java/org/apache/shardingsphere/broadcast/rule/BroadcastRule.java
b/features/broadcast/core/src/main/java/org/apache/shardingsphere/broadcast/rule/BroadcastRule.java
index a6ae174d300..110ad460eef 100644
---
a/features/broadcast/core/src/main/java/org/apache/shardingsphere/broadcast/rule/BroadcastRule.java
+++
b/features/broadcast/core/src/main/java/org/apache/shardingsphere/broadcast/rule/BroadcastRule.java
@@ -94,11 +94,6 @@ public final class BroadcastRule implements DatabaseRule,
DataNodeContainedRule,
return result;
}
- @Override
- public String getType() {
- return BroadcastRule.class.getSimpleName();
- }
-
@Override
public Map<String, Collection<DataNode>> getAllDataNodes() {
return tableDataNodes;
diff --git
a/features/encrypt/core/src/main/java/org/apache/shardingsphere/encrypt/rule/EncryptRule.java
b/features/encrypt/core/src/main/java/org/apache/shardingsphere/encrypt/rule/EncryptRule.java
index d923ba65622..4badc977b4d 100644
---
a/features/encrypt/core/src/main/java/org/apache/shardingsphere/encrypt/rule/EncryptRule.java
+++
b/features/encrypt/core/src/main/java/org/apache/shardingsphere/encrypt/rule/EncryptRule.java
@@ -172,9 +172,4 @@ public final class EncryptRule implements DatabaseRule,
TableContainedRule {
public TableNamesMapper getEnhancedTableMapper() {
return getLogicTableMapper();
}
-
- @Override
- public String getType() {
- return EncryptRule.class.getSimpleName();
- }
}
diff --git
a/features/mask/core/src/main/java/org/apache/shardingsphere/mask/rule/MaskRule.java
b/features/mask/core/src/main/java/org/apache/shardingsphere/mask/rule/MaskRule.java
index eda9e780c7f..3c41572e8ab 100644
---
a/features/mask/core/src/main/java/org/apache/shardingsphere/mask/rule/MaskRule.java
+++
b/features/mask/core/src/main/java/org/apache/shardingsphere/mask/rule/MaskRule.java
@@ -83,9 +83,4 @@ public final class MaskRule implements DatabaseRule,
TableContainedRule {
public TableNamesMapper getEnhancedTableMapper() {
return new TableNamesMapper();
}
-
- @Override
- public String getType() {
- return MaskRule.class.getSimpleName();
- }
}
diff --git
a/features/readwrite-splitting/core/src/main/java/org/apache/shardingsphere/readwritesplitting/rule/ReadwriteSplittingRule.java
b/features/readwrite-splitting/core/src/main/java/org/apache/shardingsphere/readwritesplitting/rule/ReadwriteSplittingRule.java
index 4bf5b118429..db9bad0d737 100644
---
a/features/readwrite-splitting/core/src/main/java/org/apache/shardingsphere/readwritesplitting/rule/ReadwriteSplittingRule.java
+++
b/features/readwrite-splitting/core/src/main/java/org/apache/shardingsphere/readwritesplitting/rule/ReadwriteSplittingRule.java
@@ -205,9 +205,4 @@ public final class ReadwriteSplittingRule implements
DatabaseRule, DataSourceCon
}
return result;
}
-
- @Override
- public String getType() {
- return ReadwriteSplittingRule.class.getSimpleName();
- }
}
diff --git
a/features/shadow/core/src/main/java/org/apache/shardingsphere/shadow/rule/ShadowRule.java
b/features/shadow/core/src/main/java/org/apache/shardingsphere/shadow/rule/ShadowRule.java
index 14c008df7d9..154fc28b50b 100644
---
a/features/shadow/core/src/main/java/org/apache/shardingsphere/shadow/rule/ShadowRule.java
+++
b/features/shadow/core/src/main/java/org/apache/shardingsphere/shadow/rule/ShadowRule.java
@@ -247,9 +247,4 @@ public final class ShadowRule implements DatabaseRule,
DataSourceContainedRule {
result.add(shadowDataSourceRule.getShadowDataSource());
return result;
}
-
- @Override
- public String getType() {
- return ShadowRule.class.getSimpleName();
- }
}
diff --git
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
index 4dca4fe6b81..1920fbb3c68 100644
---
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
+++
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
@@ -793,9 +793,4 @@ public final class ShardingRule implements DatabaseRule,
DataNodeContainedRule,
public boolean isShardingCacheEnabled() {
return null != shardingCache;
}
-
- @Override
- public String getType() {
- return ShardingRule.class.getSimpleName();
- }
}
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 5b88271097f..c47e90169b2 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
@@ -305,10 +305,5 @@ class CreateShardingTableRuleStatementUpdaterTest {
public Map<String, Collection<String>> getDataSourceMapper() {
return Collections.singletonMap("logic_ds", null);
}
-
- @Override
- public String getType() {
- return "mock";
- }
}
}
diff --git
a/infra/common/src/main/java/org/apache/shardingsphere/infra/rule/ShardingSphereRule.java
b/infra/common/src/main/java/org/apache/shardingsphere/infra/rule/ShardingSphereRule.java
index ac887ac5491..4102ec08de9 100644
---
a/infra/common/src/main/java/org/apache/shardingsphere/infra/rule/ShardingSphereRule.java
+++
b/infra/common/src/main/java/org/apache/shardingsphere/infra/rule/ShardingSphereRule.java
@@ -30,11 +30,4 @@ public interface ShardingSphereRule {
* @return rule configuration
*/
RuleConfiguration getConfiguration();
-
- /**
- * Get type.
- *
- * @return rule type
- */
- String getType();
}
diff --git
a/infra/common/src/test/java/org/apache/shardingsphere/infra/fixture/FixtureRule.java
b/infra/common/src/test/java/org/apache/shardingsphere/infra/fixture/FixtureRule.java
index e50265e9287..35a9971bd98 100644
---
a/infra/common/src/test/java/org/apache/shardingsphere/infra/fixture/FixtureRule.java
+++
b/infra/common/src/test/java/org/apache/shardingsphere/infra/fixture/FixtureRule.java
@@ -38,9 +38,4 @@ public final class FixtureRule implements DatabaseRule,
DataSourceContainedRule
public Map<String, Collection<String>> getDataSourceMapper() {
return Collections.emptyMap();
}
-
- @Override
- public String getType() {
- return FixtureRule.class.getSimpleName();
- }
}
diff --git
a/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/rule/ShardingSphereRuleFixture.java
b/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/rule/ShardingSphereRuleFixture.java
index 48a3211a0f4..67f6ca44944 100644
---
a/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/rule/ShardingSphereRuleFixture.java
+++
b/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/rule/ShardingSphereRuleFixture.java
@@ -28,9 +28,4 @@ public class ShardingSphereRuleFixture implements
ShardingSphereRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return ShardingSphereRuleFixture.class.getSimpleName();
- }
}
diff --git
a/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/fixture/rule/TableContainedFixtureRule.java
b/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/fixture/rule/TableContainedFixtureRule.java
index dac17fbc6be..91251a632ed 100644
---
a/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/fixture/rule/TableContainedFixtureRule.java
+++
b/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/fixture/rule/TableContainedFixtureRule.java
@@ -49,9 +49,4 @@ public final class TableContainedFixtureRule implements
TableContainedRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return TableContainedFixtureRule.class.getSimpleName();
- }
}
diff --git
a/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureDatabaseRule.java
b/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureDatabaseRule.java
index ecda50857b3..86775ce9c2a 100644
---
a/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureDatabaseRule.java
+++
b/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureDatabaseRule.java
@@ -28,9 +28,4 @@ public final class FixtureDatabaseRule implements
DatabaseRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return FixtureDatabaseRule.class.getSimpleName();
- }
}
diff --git
a/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureGlobalRule.java
b/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureGlobalRule.java
index 669c36071c5..c365e985578 100644
---
a/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureGlobalRule.java
+++
b/infra/common/src/test/java/org/apache/shardingsphere/infra/rule/builder/fixture/FixtureGlobalRule.java
@@ -28,9 +28,4 @@ public final class FixtureGlobalRule implements GlobalRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return FixtureGlobalRule.class.getSimpleName();
- }
}
diff --git
a/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/DecoratorRuleFixture.java
b/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/DecoratorRuleFixture.java
index 8873285157c..6e2b3cdaa8c 100644
---
a/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/DecoratorRuleFixture.java
+++
b/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/DecoratorRuleFixture.java
@@ -28,9 +28,4 @@ public final class DecoratorRuleFixture implements
ShardingSphereRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return DecoratorRuleFixture.class.getSimpleName();
- }
}
diff --git
a/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/MergerRuleFixture.java
b/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/MergerRuleFixture.java
index de9916edddf..6d15a2d293b 100644
---
a/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/MergerRuleFixture.java
+++
b/infra/merge/src/test/java/org/apache/shardingsphere/infra/merge/fixture/rule/MergerRuleFixture.java
@@ -28,9 +28,4 @@ public final class MergerRuleFixture implements
ShardingSphereRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return MergerRuleFixture.class.getSimpleName();
- }
}
diff --git
a/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteFailureRuleFixture.java
b/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteFailureRuleFixture.java
index 123c1f4b384..3e0c24fea42 100644
---
a/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteFailureRuleFixture.java
+++
b/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteFailureRuleFixture.java
@@ -28,9 +28,4 @@ public final class RouteFailureRuleFixture implements
ShardingSphereRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return RouteFailureRuleFixture.class.getSimpleName();
- }
}
diff --git
a/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteRuleFixture.java
b/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteRuleFixture.java
index 5b7552395be..edb1c534dcf 100644
---
a/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteRuleFixture.java
+++
b/infra/route/src/test/java/org/apache/shardingsphere/infra/route/fixture/rule/RouteRuleFixture.java
@@ -28,9 +28,4 @@ public final class RouteRuleFixture implements
ShardingSphereRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return RouteRuleFixture.class.getSimpleName();
- }
}
diff --git
a/kernel/authority/core/src/main/java/org/apache/shardingsphere/authority/rule/AuthorityRule.java
b/kernel/authority/core/src/main/java/org/apache/shardingsphere/authority/rule/AuthorityRule.java
index 134ec950a90..9f1fdb0a82a 100644
---
a/kernel/authority/core/src/main/java/org/apache/shardingsphere/authority/rule/AuthorityRule.java
+++
b/kernel/authority/core/src/main/java/org/apache/shardingsphere/authority/rule/AuthorityRule.java
@@ -76,9 +76,4 @@ public final class AuthorityRule implements GlobalRule {
public Optional<ShardingSpherePrivileges> findPrivileges(final Grantee
grantee) {
return authorityRegistry.findPrivileges(grantee);
}
-
- @Override
- public String getType() {
- return AuthorityRule.class.getSimpleName();
- }
}
diff --git
a/kernel/global-clock/core/src/main/java/org/apache/shardingsphere/globalclock/core/rule/GlobalClockRule.java
b/kernel/global-clock/core/src/main/java/org/apache/shardingsphere/globalclock/core/rule/GlobalClockRule.java
index 13d5b9ca101..7703ae0af3d 100644
---
a/kernel/global-clock/core/src/main/java/org/apache/shardingsphere/globalclock/core/rule/GlobalClockRule.java
+++
b/kernel/global-clock/core/src/main/java/org/apache/shardingsphere/globalclock/core/rule/GlobalClockRule.java
@@ -72,9 +72,4 @@ public final class GlobalClockRule implements GlobalRule {
public String getGlobalClockProviderType() {
return String.join(".", configuration.getType(),
configuration.getProvider());
}
-
- @Override
- public String getType() {
- return GlobalClockRule.class.getSimpleName();
- }
}
diff --git
a/kernel/logging/core/src/main/java/org/apache/shardingsphere/logging/rule/LoggingRule.java
b/kernel/logging/core/src/main/java/org/apache/shardingsphere/logging/rule/LoggingRule.java
index 7461dcdbe77..8148e5fb3e9 100644
---
a/kernel/logging/core/src/main/java/org/apache/shardingsphere/logging/rule/LoggingRule.java
+++
b/kernel/logging/core/src/main/java/org/apache/shardingsphere/logging/rule/LoggingRule.java
@@ -32,9 +32,4 @@ public final class LoggingRule implements GlobalRule {
public LoggingRule(final LoggingRuleConfiguration ruleConfig) {
configuration = ruleConfig;
}
-
- @Override
- public String getType() {
- return LoggingRule.class.getSimpleName();
- }
}
diff --git
a/kernel/logging/core/src/test/java/org/apache/shardingsphere/logging/rule/LoggingRuleTest.java
b/kernel/logging/core/src/test/java/org/apache/shardingsphere/logging/rule/LoggingRuleTest.java
index ce0031467e1..17ff3eb3a79 100644
---
a/kernel/logging/core/src/test/java/org/apache/shardingsphere/logging/rule/LoggingRuleTest.java
+++
b/kernel/logging/core/src/test/java/org/apache/shardingsphere/logging/rule/LoggingRuleTest.java
@@ -38,11 +38,6 @@ class LoggingRuleTest {
Collections.singleton(new ShardingSphereAppender("console",
"ch.qos.logback.core.ConsoleAppender", "[%-5level] %d{yyyy-MM-dd HH:mm:ss.SSS}
[%thread] %logger{36} - %msg%n"))));
}
- @Test
- void assertGetType() {
- assertThat(loggingRule.getType(),
is(LoggingRule.class.getSimpleName()));
- }
-
@Test
void assertFields() {
assertThat(loggingRule.getConfiguration().getLoggers().size(), is(1));
diff --git
a/kernel/single/core/src/main/java/org/apache/shardingsphere/single/rule/SingleRule.java
b/kernel/single/core/src/main/java/org/apache/shardingsphere/single/rule/SingleRule.java
index c3b7908fdd1..9394b796bcb 100644
---
a/kernel/single/core/src/main/java/org/apache/shardingsphere/single/rule/SingleRule.java
+++
b/kernel/single/core/src/main/java/org/apache/shardingsphere/single/rule/SingleRule.java
@@ -309,9 +309,4 @@ public final class SingleRule implements DatabaseRule,
DataNodeContainedRule, Ta
public Map<String, Object> getExportData() {
return
Collections.singletonMap(ExportableConstants.EXPORT_SINGLE_TABLES,
tableNamesMapper.getTableNames());
}
-
- @Override
- public String getType() {
- return SingleRule.class.getSimpleName();
- }
}
diff --git
a/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/rule/SQLFederationRule.java
b/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/rule/SQLFederationRule.java
index b2ba12064d6..af866624c59 100644
---
a/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/rule/SQLFederationRule.java
+++
b/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/rule/SQLFederationRule.java
@@ -62,9 +62,4 @@ public final class SQLFederationRule implements GlobalRule,
MetaDataHeldRule {
optimizerContext.removeParserContext(databaseName);
optimizerContext.removePlannerContext(databaseName);
}
-
- @Override
- public String getType() {
- return SQLFederationRule.class.getSimpleName();
- }
}
diff --git
a/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleMatchFixture.java
b/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleMatchFixture.java
index 25dad8dbcd9..fdbce39cfc6 100644
---
a/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleMatchFixture.java
+++
b/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleMatchFixture.java
@@ -30,9 +30,4 @@ public final class SQLFederationDeciderRuleMatchFixture
implements ShardingSpher
public RuleConfiguration getConfiguration() {
return new SQLFederationRuleConfiguration(true,
mock(CacheOption.class));
}
-
- @Override
- public String getType() {
- return SQLFederationDeciderRuleMatchFixture.class.getSimpleName();
- }
}
diff --git
a/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleNotMatchFixture.java
b/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleNotMatchFixture.java
index 36788a0f27b..e32f797264a 100644
---
a/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleNotMatchFixture.java
+++
b/kernel/sql-federation/core/src/test/java/org/apache/shardingsphere/sqlfederation/engine/fixture/rule/SQLFederationDeciderRuleNotMatchFixture.java
@@ -30,9 +30,4 @@ public final class SQLFederationDeciderRuleNotMatchFixture
implements ShardingSp
public RuleConfiguration getConfiguration() {
return new SQLFederationRuleConfiguration(false,
mock(CacheOption.class));
}
-
- @Override
- public String getType() {
- return SQLFederationDeciderRuleNotMatchFixture.class.getSimpleName();
- }
}
diff --git
a/kernel/sql-parser/core/src/main/java/org/apache/shardingsphere/parser/rule/SQLParserRule.java
b/kernel/sql-parser/core/src/main/java/org/apache/shardingsphere/parser/rule/SQLParserRule.java
index 6d8cdb7002e..d9b16552854 100644
---
a/kernel/sql-parser/core/src/main/java/org/apache/shardingsphere/parser/rule/SQLParserRule.java
+++
b/kernel/sql-parser/core/src/main/java/org/apache/shardingsphere/parser/rule/SQLParserRule.java
@@ -61,9 +61,4 @@ public final class SQLParserRule implements GlobalRule {
? new ShardingSphereSQLParserEngine(databaseType,
sqlStatementCache, parseTreeCache, sqlCommentParseEnabled)
: new SimpleSQLParserEngine();
}
-
- @Override
- public String getType() {
- return SQLParserRule.class.getSimpleName();
- }
}
diff --git
a/kernel/sql-parser/core/src/test/java/org/apache/shardingsphere/parser/rule/SQLParserRuleTest.java
b/kernel/sql-parser/core/src/test/java/org/apache/shardingsphere/parser/rule/SQLParserRuleTest.java
index 9c5734c13e7..e4592a973d7 100644
---
a/kernel/sql-parser/core/src/test/java/org/apache/shardingsphere/parser/rule/SQLParserRuleTest.java
+++
b/kernel/sql-parser/core/src/test/java/org/apache/shardingsphere/parser/rule/SQLParserRuleTest.java
@@ -43,11 +43,6 @@ class SQLParserRuleTest {
assertNotNull(sqlParserRule.getSQLParserEngine(TypedSPILoader.getService(DatabaseType.class,
"H2")));
}
- @Test
- void assertGetType() {
- assertThat(sqlParserRule.getType(),
is(SQLParserRule.class.getSimpleName()));
- }
-
@Test
void assertFields() {
assertTrue(sqlParserRule.isSqlCommentParseEnabled());
diff --git
a/kernel/sql-translator/core/src/main/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRule.java
b/kernel/sql-translator/core/src/main/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRule.java
index 272eea1efc9..3355e010626 100644
---
a/kernel/sql-translator/core/src/main/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRule.java
+++
b/kernel/sql-translator/core/src/main/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRule.java
@@ -66,9 +66,4 @@ public final class SQLTranslatorRule implements GlobalRule {
throw ex;
}
}
-
- @Override
- public String getType() {
- return SQLTranslatorRule.class.getSimpleName();
- }
}
diff --git
a/kernel/sql-translator/core/src/test/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRuleTest.java
b/kernel/sql-translator/core/src/test/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRuleTest.java
index b4d6aece840..6d847167a08 100644
---
a/kernel/sql-translator/core/src/test/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRuleTest.java
+++
b/kernel/sql-translator/core/src/test/java/org/apache/shardingsphere/sqltranslator/rule/SQLTranslatorRuleTest.java
@@ -75,9 +75,4 @@ class SQLTranslatorRuleTest {
SQLTranslatorRuleConfiguration expected = new
SQLTranslatorRuleConfiguration("CONVERT_TO_UPPER_CASE", false);
assertThat(new SQLTranslatorRule(expected).getConfiguration(),
is(expected));
}
-
- @Test
- void assertGetType() {
- assertThat(new SQLTranslatorRule(new
SQLTranslatorRuleConfiguration("CONVERT_TO_UPPER_CASE", false)).getType(),
is(SQLTranslatorRule.class.getSimpleName()));
- }
}
diff --git
a/kernel/time-service/core/src/main/java/org/apache/shardingsphere/timeservice/core/rule/TimestampServiceRule.java
b/kernel/time-service/core/src/main/java/org/apache/shardingsphere/timeservice/core/rule/TimestampServiceRule.java
index 8a3df4f71a4..87fb65f3f77 100644
---
a/kernel/time-service/core/src/main/java/org/apache/shardingsphere/timeservice/core/rule/TimestampServiceRule.java
+++
b/kernel/time-service/core/src/main/java/org/apache/shardingsphere/timeservice/core/rule/TimestampServiceRule.java
@@ -49,9 +49,4 @@ public final class TimestampServiceRule implements GlobalRule
{
public Timestamp getTimestamp() {
return timestampService.getTimestamp();
}
-
- @Override
- public String getType() {
- return TimestampServiceRule.class.getSimpleName();
- }
}
diff --git
a/kernel/traffic/core/src/main/java/org/apache/shardingsphere/traffic/rule/TrafficRule.java
b/kernel/traffic/core/src/main/java/org/apache/shardingsphere/traffic/rule/TrafficRule.java
index 24b5a47374a..cdca0f4e8a7 100644
---
a/kernel/traffic/core/src/main/java/org/apache/shardingsphere/traffic/rule/TrafficRule.java
+++
b/kernel/traffic/core/src/main/java/org/apache/shardingsphere/traffic/rule/TrafficRule.java
@@ -189,9 +189,4 @@ public final class TrafficRule implements GlobalRule {
}
return result;
}
-
- @Override
- public String getType() {
- return TrafficRule.class.getSimpleName();
- }
}
diff --git
a/kernel/transaction/core/src/main/java/org/apache/shardingsphere/transaction/rule/TransactionRule.java
b/kernel/transaction/core/src/main/java/org/apache/shardingsphere/transaction/rule/TransactionRule.java
index 63dc7914a74..90529dbe0bd 100644
---
a/kernel/transaction/core/src/main/java/org/apache/shardingsphere/transaction/rule/TransactionRule.java
+++
b/kernel/transaction/core/src/main/java/org/apache/shardingsphere/transaction/rule/TransactionRule.java
@@ -141,9 +141,4 @@ public final class TransactionRule implements GlobalRule,
ResourceHeldRule<Shard
log.error("Close transaction engine failed", ex);
}
}
-
- @Override
- public String getType() {
- return TransactionRule.class.getSimpleName();
- }
}
diff --git
a/mode/core/src/test/java/org/apache/shardingsphere/mode/fixture/FixtureDatabaseRule.java
b/mode/core/src/test/java/org/apache/shardingsphere/mode/fixture/FixtureDatabaseRule.java
index 99e158cf506..9529c462968 100644
---
a/mode/core/src/test/java/org/apache/shardingsphere/mode/fixture/FixtureDatabaseRule.java
+++
b/mode/core/src/test/java/org/apache/shardingsphere/mode/fixture/FixtureDatabaseRule.java
@@ -31,9 +31,4 @@ public final class FixtureDatabaseRule implements
DatabaseRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return FixtureDatabaseRule.class.getSimpleName();
- }
}
diff --git
a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/rdl/storage/unit/UnregisterStorageUnitBackendHandlerTest.java
b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/rdl/storage/unit/UnregisterStorageUnitBackendHandlerTest.java
index dc155e15b4a..3812b371fa6 100644
---
a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/rdl/storage/unit/UnregisterStorageUnitBackendHandlerTest.java
+++
b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/distsql/rdl/storage/unit/UnregisterStorageUnitBackendHandlerTest.java
@@ -130,7 +130,6 @@ class UnregisterStorageUnitBackendHandlerTest {
@Test
void assertStorageUnitNameInUseExecute() {
when(ruleMetaData.findRules(DataSourceContainedRule.class)).thenReturn(Collections.singleton(shadowRule));
- when(shadowRule.getType()).thenReturn("ShadowRule");
when(shadowRule.getDataSourceMapper()).thenReturn(Collections.singletonMap("",
Collections.singleton("foo_ds")));
StorageUnit storageUnit = mock(StorageUnit.class);
when(storageUnit.getDataSource()).thenReturn(dataSource);
@@ -144,7 +143,6 @@ class UnregisterStorageUnitBackendHandlerTest {
@Test
void assertStorageUnitNameInUseWithoutIgnoreSingleTables() {
when(ruleMetaData.findRules(DataNodeContainedRule.class)).thenReturn(Collections.singleton(singleRule));
- when(singleRule.getType()).thenReturn("SingleRule");
DataNode dataNode = mock(DataNode.class);
when(dataNode.getDataSourceName()).thenReturn("foo_ds");
when(singleRule.getAllDataNodes()).thenReturn(Collections.singletonMap("",
Collections.singleton(dataNode)));
@@ -160,7 +158,6 @@ class UnregisterStorageUnitBackendHandlerTest {
@Test
void assertStorageUnitNameInUseIgnoreSingleTables() throws SQLException {
when(ruleMetaData.findRules(DataNodeContainedRule.class)).thenReturn(Collections.singleton(singleRule));
- when(singleRule.getType()).thenReturn("SingleRule");
DataNode dataNode = mock(DataNode.class);
when(dataNode.getDataSourceName()).thenReturn("foo_ds");
when(singleRule.getAllDataNodes()).thenReturn(Collections.singletonMap("",
Collections.singleton(dataNode)));
@@ -184,7 +181,6 @@ class UnregisterStorageUnitBackendHandlerTest {
@Test
void assertStorageUnitNameInUseWithIfExists() {
when(ruleMetaData.findRules(DataSourceContainedRule.class)).thenReturn(Collections.singleton(shadowRule));
- when(shadowRule.getType()).thenReturn("ShadowRule");
when(shadowRule.getDataSourceMapper()).thenReturn(Collections.singletonMap("",
Collections.singleton("foo_ds")));
when(contextManager.getMetaDataContexts().getMetaData().getDatabase("foo_db")).thenReturn(database);
UnregisterStorageUnitStatement unregisterStorageUnitStatement = new
UnregisterStorageUnitStatement(true, Collections.singleton("foo_ds"), true);
diff --git
a/test/fixture/infra/src/main/java/org/apache/shardingsphere/test/fixture/infra/rule/MockedRule.java
b/test/fixture/infra/src/main/java/org/apache/shardingsphere/test/fixture/infra/rule/MockedRule.java
index b5db7812574..0425cec047c 100644
---
a/test/fixture/infra/src/main/java/org/apache/shardingsphere/test/fixture/infra/rule/MockedRule.java
+++
b/test/fixture/infra/src/main/java/org/apache/shardingsphere/test/fixture/infra/rule/MockedRule.java
@@ -31,9 +31,4 @@ public final class MockedRule implements ShardingSphereRule {
public RuleConfiguration getConfiguration() {
return mock(RuleConfiguration.class);
}
-
- @Override
- public String getType() {
- return MockedRule.class.getSimpleName();
- }
}
