You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by zh...@apache.org on 2025/10/08 10:13:58 UTC
(shardingsphere) branch master updated: Make inner classes static for better memory management (#36819)
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 7abf4cdb4a3 Make inner classes static for better memory management (#36819)
7abf4cdb4a3 is described below
commit 7abf4cdb4a3f74ac34d0bc6156c38287bacd6c8f
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Wed Oct 8 18:13:52 2025 +0800
Make inner classes static for better memory management (#36819)
---
.../connector/core/jdbcurl/parser/StandardJdbcUrlParserTest.java | 2 +-
.../jdbcurl/ClickHouseConnectionPropertiesParserTest.java | 2 +-
.../firebird/jdbcurl/FirebirdConnectionPropertiesParserTest.java | 2 +-
.../database/connector/h2/jdbcurl/H2DatabaseInstanceJudgerTest.java | 2 +-
.../connector/hive/jdbcurl/HiveConnectionPropertiesParserTest.java | 2 +-
.../mysql/jdbcurl/MySQLConnectionPropertiesParserTest.java | 2 +-
.../opengauss/jdbcurl/OpenGaussConnectionPropertiesParserTest.java | 2 +-
.../oracle/jdbcurl/OracleConnectionPropertiesParserTest.java | 2 +-
.../jdbcurl/PostgreSQLConnectionPropertiesParserTest.java | 2 +-
.../sql92/jdbcurl/SQL92ConnectionPropertiesParserTest.java | 2 +-
.../sqlserver/jdbcurl/SQLServerConnectionPropertiesParserTest.java | 2 +-
.../exception/mysql/mapper/MySQLDialectExceptionMapperTest.java | 2 +-
.../exception/postgresql/PostgreSQLDialectExceptionMapperTest.java | 2 +-
.../protocol/mysql/netty/MySQLSequenceIdInboundHandlerTest.java | 4 ++--
.../row/column/value/integer/MySQLBitBinlogProtocolValueTest.java | 2 +-
.../bind/protocol/PostgreSQLNumericBinaryProtocolValueTest.java | 2 +-
.../bind/protocol/text/impl/PostgreSQLTimeValueParserTest.java | 2 +-
.../bind/protocol/text/impl/PostgreSQLTimestampValueParserTest.java | 2 +-
.../packet/generic/PostgreSQLCommandCompletePacketTest.java | 2 +-
.../org/apache/shardingsphere/encrypt/rule/EncryptRuleTest.java | 2 +-
.../mask/algorithm/cover/KeepFirstNLastMMaskAlgorithmTest.java | 6 +++---
.../mask/algorithm/cover/KeepFromXToYMaskAlgorithmTest.java | 6 +++---
.../mask/algorithm/cover/MaskAfterSpecialCharsAlgorithmTest.java | 4 ++--
.../mask/algorithm/cover/MaskBeforeSpecialCharsAlgorithmTest.java | 4 ++--
.../mask/algorithm/cover/MaskFirstNLastMMaskAlgorithmTest.java | 6 +++---
.../mask/algorithm/cover/MaskFromXToYMaskAlgorithmTest.java | 6 +++---
.../shardingsphere/mask/algorithm/hash/MD5MaskAlgorithmTest.java | 4 ++--
.../handler/query/ShowDefaultShadowAlgorithmExecutorTest.java | 2 +-
.../distsql/handler/query/ShowShadowAlgorithmsExecutorTest.java | 2 +-
.../shadow/distsql/handler/query/ShowShadowRuleExecutorTest.java | 2 +-
.../distsql/handler/query/ShowShadowTableRulesExecutorTest.java | 2 +-
.../sharding/cache/checker/ShardingRouteCacheableCheckerTest.java | 2 +-
.../sharding/merge/common/IteratorStreamMergedResultTest.java | 2 +-
.../sharding/merge/ddl/fetch/FetchStreamMergedResultTest.java | 2 +-
.../sharding/route/engine/type/standard/SQLRouteTest.java | 2 +-
.../sharding/route/engine/type/standard/SubqueryRouteTest.java | 2 +-
.../exception/external/sql/ShardingSphereSQLExceptionTest.java | 2 +-
.../apache/shardingsphere/infra/url/core/ShardingSphereURLTest.java | 2 +-
.../api/type/ShardingSpherePipelineDataSourceConfiguration.java | 2 +-
.../data/pipeline/core/util/PipelineDistributedBarrier.java | 2 +-
.../core/execute/AbstractPipelineLifecycleRunnableTest.java | 2 +-
.../distsql/handler/update/AlterGlobalClockRuleExecutorTest.java | 2 +-
.../sqlfederation/compiler/compiler/it/SQLStatementCompilerIT.java | 2 +-
.../distsql/handler/update/AlterSQLFederationRuleExecutorTest.java | 2 +-
.../distsql/handler/update/AlterSQLParserRuleExecutorTest.java | 2 +-
.../distsql/handler/update/AlterSQLTranslatorRuleExecutorTest.java | 2 +-
.../distsql/handler/update/AlterTransactionRuleExecutorTest.java | 2 +-
.../mode/node/path/engine/generator/NodePathSegmentTest.java | 2 +-
.../statement/core/extractor/ColumnExtractorParameterizedTest.java | 2 +-
.../proxy/backend/session/ServerPreparedStatementRegistryTest.java | 2 +-
.../authentication/authenticator/FirebirdAuthenticatorTypeTest.java | 2 +-
.../authentication/authenticator/MySQLAuthenticatorTypeTest.java | 2 +-
.../authenticator/OpenGaussAuthenticatorTypeTest.java | 2 +-
.../authenticator/PostgreSQLAuthenticatorTypeTest.java | 2 +-
.../cases/alterresource/SwitchingTransactionRuleTestCase.java | 4 ++--
.../transaction/cases/classictransfer/ClassicTransferTestCase.java | 2 +-
.../shardingsphere/test/e2e/sql/env/DataSetEnvironmentManager.java | 4 ++--
.../framework/matcher/ShardingSphereArgumentVerifyMatchers.java | 2 +-
.../infra/framework/matcher/ShardingSphereAssertionMatchers.java | 2 +-
.../org/apache/shardingsphere/test/it/sql/binder/SQLBinderIT.java | 2 +-
.../it/optimizer/sqlnode/converter/SQLNodeConverterEngineIT.java | 2 +-
.../test/it/sql/parser/external/ExternalSQLParserIT.java | 2 +-
.../test/it/sql/parser/internal/InternalSQLParserIT.java | 2 +-
.../test/it/sql/parser/internal/InternalUnsupportedSQLParserIT.java | 2 +-
.../shardingsphere/test/it/rewriter/engine/SQLRewriterIT.java | 2 +-
65 files changed, 79 insertions(+), 79 deletions(-)
diff --git a/database/connector/core/src/test/java/org/apache/shardingsphere/database/connector/core/jdbcurl/parser/StandardJdbcUrlParserTest.java b/database/connector/core/src/test/java/org/apache/shardingsphere/database/connector/core/jdbcurl/parser/StandardJdbcUrlParserTest.java
index bb43715cb76..28b85805dc3 100644
--- a/database/connector/core/src/test/java/org/apache/shardingsphere/database/connector/core/jdbcurl/parser/StandardJdbcUrlParserTest.java
+++ b/database/connector/core/src/test/java/org/apache/shardingsphere/database/connector/core/jdbcurl/parser/StandardJdbcUrlParserTest.java
@@ -54,7 +54,7 @@ class StandardJdbcUrlParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> new StandardJdbcUrlParser().parse("jdbc:h2:mem:test;DB_CLOSE_DELAY=-1;DATABASE_TO_UPPER=false;MODE=MySQL", -1));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/clickhouse/src/test/java/org/apache/shardingsphere/database/connector/clickhouse/jdbcurl/ClickHouseConnectionPropertiesParserTest.java b/database/connector/dialect/clickhouse/src/test/java/org/apache/shardingsphere/database/connector/clickhouse/jdbcurl/ClickHouseConnectionPropertiesParserTest.java
index accadf96218..1b5ced471d0 100644
--- a/database/connector/dialect/clickhouse/src/test/java/org/apache/shardingsphere/database/connector/clickhouse/jdbcurl/ClickHouseConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/clickhouse/src/test/java/org/apache/shardingsphere/database/connector/clickhouse/jdbcurl/ClickHouseConnectionPropertiesParserTest.java
@@ -60,7 +60,7 @@ class ClickHouseConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("jdbc:ch:xxxxxxxx", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/firebird/src/test/java/org/apache/shardingsphere/database/connector/firebird/jdbcurl/FirebirdConnectionPropertiesParserTest.java b/database/connector/dialect/firebird/src/test/java/org/apache/shardingsphere/database/connector/firebird/jdbcurl/FirebirdConnectionPropertiesParserTest.java
index 8b535c347f5..09c1c483405 100644
--- a/database/connector/dialect/firebird/src/test/java/org/apache/shardingsphere/database/connector/firebird/jdbcurl/FirebirdConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/firebird/src/test/java/org/apache/shardingsphere/database/connector/firebird/jdbcurl/FirebirdConnectionPropertiesParserTest.java
@@ -62,7 +62,7 @@ class FirebirdConnectionPropertiesParserTest {
assertThrows(SQLNonTransientConnectionException.class, () -> parser.parse("jdbc:firebirdsql://localhost:c:/data/db/test.fdb", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/h2/src/test/java/org/apache/shardingsphere/database/connector/h2/jdbcurl/H2DatabaseInstanceJudgerTest.java b/database/connector/dialect/h2/src/test/java/org/apache/shardingsphere/database/connector/h2/jdbcurl/H2DatabaseInstanceJudgerTest.java
index 54bbfbd59bc..f20771a4330 100644
--- a/database/connector/dialect/h2/src/test/java/org/apache/shardingsphere/database/connector/h2/jdbcurl/H2DatabaseInstanceJudgerTest.java
+++ b/database/connector/dialect/h2/src/test/java/org/apache/shardingsphere/database/connector/h2/jdbcurl/H2DatabaseInstanceJudgerTest.java
@@ -49,7 +49,7 @@ class H2DatabaseInstanceJudgerTest {
assertThat(judger.isInSameDatabaseInstance(actual1, actual2), is(isSame));
}
- private static class IsInSameDatabaseInstanceTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class IsInSameDatabaseInstanceTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/hive/src/test/java/org/apache/shardingsphere/database/connector/hive/jdbcurl/HiveConnectionPropertiesParserTest.java b/database/connector/dialect/hive/src/test/java/org/apache/shardingsphere/database/connector/hive/jdbcurl/HiveConnectionPropertiesParserTest.java
index ffa696a5f42..ab8edfe0810 100644
--- a/database/connector/dialect/hive/src/test/java/org/apache/shardingsphere/database/connector/hive/jdbcurl/HiveConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/hive/src/test/java/org/apache/shardingsphere/database/connector/hive/jdbcurl/HiveConnectionPropertiesParserTest.java
@@ -63,7 +63,7 @@ class HiveConnectionPropertiesParserTest {
assertThrows(RuntimeException.class, () -> parser.parse("jdbc:hive2://", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/mysql/src/test/java/org/apache/shardingsphere/database/connector/mysql/jdbcurl/MySQLConnectionPropertiesParserTest.java b/database/connector/dialect/mysql/src/test/java/org/apache/shardingsphere/database/connector/mysql/jdbcurl/MySQLConnectionPropertiesParserTest.java
index 2153e6c8b36..862268707cc 100644
--- a/database/connector/dialect/mysql/src/test/java/org/apache/shardingsphere/database/connector/mysql/jdbcurl/MySQLConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/mysql/src/test/java/org/apache/shardingsphere/database/connector/mysql/jdbcurl/MySQLConnectionPropertiesParserTest.java
@@ -60,7 +60,7 @@ class MySQLConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("jdbc:mysql:xxxxxxxx", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/opengauss/src/test/java/org/apache/shardingsphere/database/connector/opengauss/jdbcurl/OpenGaussConnectionPropertiesParserTest.java b/database/connector/dialect/opengauss/src/test/java/org/apache/shardingsphere/database/connector/opengauss/jdbcurl/OpenGaussConnectionPropertiesParserTest.java
index 3443e95bdc5..73a5ad88304 100644
--- a/database/connector/dialect/opengauss/src/test/java/org/apache/shardingsphere/database/connector/opengauss/jdbcurl/OpenGaussConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/opengauss/src/test/java/org/apache/shardingsphere/database/connector/opengauss/jdbcurl/OpenGaussConnectionPropertiesParserTest.java
@@ -60,7 +60,7 @@ class OpenGaussConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("jdbc:opengauss:xxxxxxxx", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/oracle/src/test/java/org/apache/shardingsphere/database/connector/oracle/jdbcurl/OracleConnectionPropertiesParserTest.java b/database/connector/dialect/oracle/src/test/java/org/apache/shardingsphere/database/connector/oracle/jdbcurl/OracleConnectionPropertiesParserTest.java
index 436cee42de1..be7ce30590a 100644
--- a/database/connector/dialect/oracle/src/test/java/org/apache/shardingsphere/database/connector/oracle/jdbcurl/OracleConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/oracle/src/test/java/org/apache/shardingsphere/database/connector/oracle/jdbcurl/OracleConnectionPropertiesParserTest.java
@@ -58,7 +58,7 @@ class OracleConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("jdbc:oracle:xxxxxxxx", "test", null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/connector/postgresql/jdbcurl/PostgreSQLConnectionPropertiesParserTest.java b/database/connector/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/connector/postgresql/jdbcurl/PostgreSQLConnectionPropertiesParserTest.java
index 107e40854d9..8715fc432a2 100644
--- a/database/connector/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/connector/postgresql/jdbcurl/PostgreSQLConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/connector/postgresql/jdbcurl/PostgreSQLConnectionPropertiesParserTest.java
@@ -60,7 +60,7 @@ class PostgreSQLConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("jdbc:postgresql:xxxxxxxx", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/sql92/src/test/java/org/apache/shardingsphere/database/connector/sql92/jdbcurl/SQL92ConnectionPropertiesParserTest.java b/database/connector/dialect/sql92/src/test/java/org/apache/shardingsphere/database/connector/sql92/jdbcurl/SQL92ConnectionPropertiesParserTest.java
index 10aec9cfd91..1abbbaf979d 100644
--- a/database/connector/dialect/sql92/src/test/java/org/apache/shardingsphere/database/connector/sql92/jdbcurl/SQL92ConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/sql92/src/test/java/org/apache/shardingsphere/database/connector/sql92/jdbcurl/SQL92ConnectionPropertiesParserTest.java
@@ -58,7 +58,7 @@ class SQL92ConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("xxx:xxxx:xxxxxxxx", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/connector/dialect/sqlserver/src/test/java/org/apache/shardingsphere/database/connector/sql92/sqlserver/jdbcurl/SQLServerConnectionPropertiesParserTest.java b/database/connector/dialect/sqlserver/src/test/java/org/apache/shardingsphere/database/connector/sql92/sqlserver/jdbcurl/SQLServerConnectionPropertiesParserTest.java
index 30884f536d8..43a32e6f010 100644
--- a/database/connector/dialect/sqlserver/src/test/java/org/apache/shardingsphere/database/connector/sql92/sqlserver/jdbcurl/SQLServerConnectionPropertiesParserTest.java
+++ b/database/connector/dialect/sqlserver/src/test/java/org/apache/shardingsphere/database/connector/sql92/sqlserver/jdbcurl/SQLServerConnectionPropertiesParserTest.java
@@ -58,7 +58,7 @@ class SQLServerConnectionPropertiesParserTest {
assertThrows(UnrecognizedDatabaseURLException.class, () -> parser.parse("jdbc:sqlserver:xxxxxxxx", null, null));
}
- private static class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class NewConstructorTestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/exception/dialect/mysql/src/test/java/org/apache/shardingsphere/database/exception/mysql/mapper/MySQLDialectExceptionMapperTest.java b/database/exception/dialect/mysql/src/test/java/org/apache/shardingsphere/database/exception/mysql/mapper/MySQLDialectExceptionMapperTest.java
index 194ed527e2b..f738a97cc8a 100644
--- a/database/exception/dialect/mysql/src/test/java/org/apache/shardingsphere/database/exception/mysql/mapper/MySQLDialectExceptionMapperTest.java
+++ b/database/exception/dialect/mysql/src/test/java/org/apache/shardingsphere/database/exception/mysql/mapper/MySQLDialectExceptionMapperTest.java
@@ -64,7 +64,7 @@ class MySQLDialectExceptionMapperTest {
assertThat(actual.getErrorCode(), is(vendorError.getVendorCode()));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/exception/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/exception/postgresql/PostgreSQLDialectExceptionMapperTest.java b/database/exception/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/exception/postgresql/PostgreSQLDialectExceptionMapperTest.java
index 485722eb96e..eea64749bc4 100644
--- a/database/exception/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/exception/postgresql/PostgreSQLDialectExceptionMapperTest.java
+++ b/database/exception/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/exception/postgresql/PostgreSQLDialectExceptionMapperTest.java
@@ -46,7 +46,7 @@ class PostgreSQLDialectExceptionMapperTest {
assertThat(new PostgreSQLDialectExceptionMapper().convert(mock(sqlDialectExceptionClazz)).getSQLState(), is(sqlState));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/netty/MySQLSequenceIdInboundHandlerTest.java b/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/netty/MySQLSequenceIdInboundHandlerTest.java
index 9e347aa3b4c..35578a30026 100644
--- a/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/netty/MySQLSequenceIdInboundHandlerTest.java
+++ b/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/netty/MySQLSequenceIdInboundHandlerTest.java
@@ -50,7 +50,7 @@ class MySQLSequenceIdInboundHandlerTest {
assertThat(channel.<ByteBuf>readOutbound().readUnsignedByte(), is((short) 1));
}
- private static class FixtureOutboundHandler extends ChannelOutboundHandlerAdapter {
+ private static final class FixtureOutboundHandler extends ChannelOutboundHandlerAdapter {
@Override
public void write(final ChannelHandlerContext context, final Object msg, final ChannelPromise promise) {
@@ -59,7 +59,7 @@ class MySQLSequenceIdInboundHandlerTest {
}
}
- private static class FixtureInboundHandler extends ChannelInboundHandlerAdapter {
+ private static final class FixtureInboundHandler extends ChannelInboundHandlerAdapter {
@Override
public void channelRead(final ChannelHandlerContext context, final Object msg) {
diff --git a/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/row/column/value/integer/MySQLBitBinlogProtocolValueTest.java b/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/row/column/value/integer/MySQLBitBinlogProtocolValueTest.java
index f01606a702b..56c3827e22c 100644
--- a/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/row/column/value/integer/MySQLBitBinlogProtocolValueTest.java
+++ b/database/protocol/dialect/mysql/src/test/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/row/column/value/integer/MySQLBitBinlogProtocolValueTest.java
@@ -60,7 +60,7 @@ class MySQLBitBinlogProtocolValueTest {
assertThat(actual.read(columnDef, payload), is(expected));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/PostgreSQLNumericBinaryProtocolValueTest.java b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/PostgreSQLNumericBinaryProtocolValueTest.java
index 03b3cf00261..63f6defe7fd 100644
--- a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/PostgreSQLNumericBinaryProtocolValueTest.java
+++ b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/PostgreSQLNumericBinaryProtocolValueTest.java
@@ -69,7 +69,7 @@ class PostgreSQLNumericBinaryProtocolValueTest {
assertThat(actualBytes, is(expected));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimeValueParserTest.java b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimeValueParserTest.java
index cb5daefc1ca..227646780bc 100644
--- a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimeValueParserTest.java
+++ b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimeValueParserTest.java
@@ -38,7 +38,7 @@ class PostgreSQLTimeValueParserTest {
assertThat(new PostgreSQLTimeValueParser().parse(input), is(expected));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimestampValueParserTest.java b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimestampValueParserTest.java
index d2d971a9dfe..d65c053cb8b 100644
--- a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimestampValueParserTest.java
+++ b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/command/query/extended/bind/protocol/text/impl/PostgreSQLTimestampValueParserTest.java
@@ -38,7 +38,7 @@ class PostgreSQLTimestampValueParserTest {
assertThat(new PostgreSQLTimestampValueParser().parse(input), is(expected));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/generic/PostgreSQLCommandCompletePacketTest.java b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/generic/PostgreSQLCommandCompletePacketTest.java
index 3caa7fa4ec8..65db2e3cedd 100644
--- a/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/generic/PostgreSQLCommandCompletePacketTest.java
+++ b/database/protocol/dialect/postgresql/src/test/java/org/apache/shardingsphere/database/protocol/postgresql/packet/generic/PostgreSQLCommandCompletePacketTest.java
@@ -48,7 +48,7 @@ class PostgreSQLCommandCompletePacketTest {
assertThat(payload.readStringNul(), is(expectedString));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/encrypt/core/src/test/java/org/apache/shardingsphere/encrypt/rule/EncryptRuleTest.java b/features/encrypt/core/src/test/java/org/apache/shardingsphere/encrypt/rule/EncryptRuleTest.java
index 83cc2d83720..009dcdf000a 100644
--- a/features/encrypt/core/src/test/java/org/apache/shardingsphere/encrypt/rule/EncryptRuleTest.java
+++ b/features/encrypt/core/src/test/java/org/apache/shardingsphere/encrypt/rule/EncryptRuleTest.java
@@ -116,7 +116,7 @@ class EncryptRuleTest {
return result;
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFirstNLastMMaskAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFirstNLastMMaskAlgorithmTest.java
index 210c94aa5ad..718174e5d6f 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFirstNLastMMaskAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFirstNLastMMaskAlgorithmTest.java
@@ -51,7 +51,7 @@ class KeepFirstNLastMMaskAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
+ private static final class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
AlgorithmInitArgumentsProvider() {
super("KEEP_FIRST_N_LAST_M");
@@ -66,7 +66,7 @@ class KeepFirstNLastMMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("KEEP_FIRST_N_LAST_M", PropertiesBuilder.build(new Property("first-n", "3"), new Property("last-m", "5"), new Property("replace-char", "*")));
@@ -87,7 +87,7 @@ class KeepFirstNLastMMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteWithSameNMArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteWithSameNMArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteWithSameNMArgumentsProvider() {
super("KEEP_FIRST_N_LAST_M", PropertiesBuilder.build(new Property("first-n", "5"), new Property("last-m", "5"), new Property("replace-char", "*")));
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFromXToYMaskAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFromXToYMaskAlgorithmTest.java
index 414d93c9662..08d9f941fcd 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFromXToYMaskAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/KeepFromXToYMaskAlgorithmTest.java
@@ -51,7 +51,7 @@ class KeepFromXToYMaskAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
+ private static final class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
AlgorithmInitArgumentsProvider() {
super("KEEP_FROM_X_TO_Y");
@@ -69,7 +69,7 @@ class KeepFromXToYMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("KEEP_FROM_X_TO_Y", PropertiesBuilder.build(new Property("from-x", "3"), new Property("to-y", "5"), new Property("replace-char", "*")));
@@ -89,7 +89,7 @@ class KeepFromXToYMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteWithSameXYArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteWithSameXYArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteWithSameXYArgumentsProvider() {
super("KEEP_FROM_X_TO_Y", PropertiesBuilder.build(new Property("from-x", "5"), new Property("to-y", "5"), new Property("replace-char", "*")));
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskAfterSpecialCharsAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskAfterSpecialCharsAlgorithmTest.java
index afbb9e61b75..cee6bed3f80 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskAfterSpecialCharsAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskAfterSpecialCharsAlgorithmTest.java
@@ -45,7 +45,7 @@ class MaskAfterSpecialCharsAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
+ private static final class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
AlgorithmInitArgumentsProvider() {
super("MASK_AFTER_SPECIAL_CHARS");
@@ -61,7 +61,7 @@ class MaskAfterSpecialCharsAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("MASK_AFTER_SPECIAL_CHARS", PropertiesBuilder.build(new Property("special-chars", "d1"), new Property("replace-char", "*")));
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskBeforeSpecialCharsAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskBeforeSpecialCharsAlgorithmTest.java
index 65aa0004817..a78d62a225b 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskBeforeSpecialCharsAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskBeforeSpecialCharsAlgorithmTest.java
@@ -45,7 +45,7 @@ class MaskBeforeSpecialCharsAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
+ private static final class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
AlgorithmInitArgumentsProvider() {
super("MASK_BEFORE_SPECIAL_CHARS");
@@ -59,7 +59,7 @@ class MaskBeforeSpecialCharsAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("MASK_BEFORE_SPECIAL_CHARS", PropertiesBuilder.build(new Property("special-chars", "d1"), new Property("replace-char", "*")));
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFirstNLastMMaskAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFirstNLastMMaskAlgorithmTest.java
index b614b495d43..c520da34e9a 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFirstNLastMMaskAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFirstNLastMMaskAlgorithmTest.java
@@ -51,7 +51,7 @@ class MaskFirstNLastMMaskAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
+ private static final class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
AlgorithmInitArgumentsProvider() {
super("MASK_FIRST_N_LAST_M");
@@ -66,7 +66,7 @@ class MaskFirstNLastMMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("MASK_FIRST_N_LAST_M", PropertiesBuilder.build(new Property("first-n", "3"), new Property("last-m", "5"), new Property("replace-char", "*")));
@@ -87,7 +87,7 @@ class MaskFirstNLastMMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteWithSameNMArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteWithSameNMArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteWithSameNMArgumentsProvider() {
super("MASK_FIRST_N_LAST_M", PropertiesBuilder.build(new Property("first-n", "5"), new Property("last-m", "5"), new Property("replace-char", "*")));
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFromXToYMaskAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFromXToYMaskAlgorithmTest.java
index 8050d25c185..ced2c5aa852 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFromXToYMaskAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/cover/MaskFromXToYMaskAlgorithmTest.java
@@ -51,7 +51,7 @@ class MaskFromXToYMaskAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
+ private static final class AlgorithmInitArgumentsProvider extends MaskAlgorithmInitArgumentsProvider {
AlgorithmInitArgumentsProvider() {
super("MASK_FROM_X_TO_Y");
@@ -69,7 +69,7 @@ class MaskFromXToYMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("MASK_FROM_X_TO_Y", PropertiesBuilder.build(new Property("from-x", "3"), new Property("to-y", "5"), new Property("replace-char", "*")));
@@ -88,7 +88,7 @@ class MaskFromXToYMaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteWithSameXYArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteWithSameXYArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteWithSameXYArgumentsProvider() {
super("MASK_FROM_X_TO_Y", PropertiesBuilder.build(new Property("from-x", "5"), new Property("to-y", "5"), new Property("replace-char", "*")));
diff --git a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/hash/MD5MaskAlgorithmTest.java b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/hash/MD5MaskAlgorithmTest.java
index 7db253ba6bf..b1e626f7cc0 100644
--- a/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/hash/MD5MaskAlgorithmTest.java
+++ b/features/mask/core/src/test/java/org/apache/shardingsphere/mask/algorithm/hash/MD5MaskAlgorithmTest.java
@@ -44,7 +44,7 @@ class MD5MaskAlgorithmTest {
MaskAlgorithmAssertions.assertMask(type, props, plainValue, maskedValue);
}
- private static class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteArgumentsProvider() {
super("MD5", new Properties());
@@ -58,7 +58,7 @@ class MD5MaskAlgorithmTest {
}
}
- private static class AlgorithmMaskExecuteWithSaltArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
+ private static final class AlgorithmMaskExecuteWithSaltArgumentsProvider extends MaskAlgorithmExecuteArgumentsProvider {
AlgorithmMaskExecuteWithSaltArgumentsProvider() {
super("MD5", PropertiesBuilder.build(new Property("salt", "202cb962ac5907")));
diff --git a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowDefaultShadowAlgorithmExecutorTest.java b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowDefaultShadowAlgorithmExecutorTest.java
index 602f6a76ec7..8937e115205 100644
--- a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowDefaultShadowAlgorithmExecutorTest.java
+++ b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowDefaultShadowAlgorithmExecutorTest.java
@@ -55,7 +55,7 @@ class ShowDefaultShadowAlgorithmExecutorTest extends DistSQLDatabaseRuleQueryExe
assertQueryResultRows(ruleConfig, sqlStatement, expected);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowAlgorithmsExecutorTest.java b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowAlgorithmsExecutorTest.java
index cdf4d9bd228..532996948ba 100644
--- a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowAlgorithmsExecutorTest.java
+++ b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowAlgorithmsExecutorTest.java
@@ -55,7 +55,7 @@ class ShowShadowAlgorithmsExecutorTest extends DistSQLDatabaseRuleQueryExecutorT
assertQueryResultRows(ruleConfig, sqlStatement, expected);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowRuleExecutorTest.java b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowRuleExecutorTest.java
index 5a7e1b02ea6..b6290fe044b 100644
--- a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowRuleExecutorTest.java
+++ b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowRuleExecutorTest.java
@@ -56,7 +56,7 @@ class ShowShadowRuleExecutorTest extends DistSQLDatabaseRuleQueryExecutorTest {
assertQueryResultRows(ruleConfig, sqlStatement, expected);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowTableRulesExecutorTest.java b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowTableRulesExecutorTest.java
index 3017df81b51..b740513aaba 100644
--- a/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowTableRulesExecutorTest.java
+++ b/features/shadow/distsql/handler/src/test/java/org/apache/shardingsphere/shadow/distsql/handler/query/ShowShadowTableRulesExecutorTest.java
@@ -56,7 +56,7 @@ class ShowShadowTableRulesExecutorTest extends DistSQLDatabaseRuleQueryExecutorT
assertQueryResultRows(ruleConfig, sqlStatement, expected);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/cache/checker/ShardingRouteCacheableCheckerTest.java b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/cache/checker/ShardingRouteCacheableCheckerTest.java
index ab6581df4a8..0aab108020f 100644
--- a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/cache/checker/ShardingRouteCacheableCheckerTest.java
+++ b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/cache/checker/ShardingRouteCacheableCheckerTest.java
@@ -168,7 +168,7 @@ class ShardingRouteCacheableCheckerTest {
return sqlParserRule.getSQLParserEngine(databaseType).parse(sql, false);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/common/IteratorStreamMergedResultTest.java b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/common/IteratorStreamMergedResultTest.java
index 6d1580ad774..4ba9c43d64d 100644
--- a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/common/IteratorStreamMergedResultTest.java
+++ b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/common/IteratorStreamMergedResultTest.java
@@ -127,7 +127,7 @@ class IteratorStreamMergedResultTest {
assertFalse(actual.next());
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/ddl/fetch/FetchStreamMergedResultTest.java b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/ddl/fetch/FetchStreamMergedResultTest.java
index 1f2552723c7..7b64e6afbef 100644
--- a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/ddl/fetch/FetchStreamMergedResultTest.java
+++ b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/merge/ddl/fetch/FetchStreamMergedResultTest.java
@@ -167,7 +167,7 @@ class FetchStreamMergedResultTest {
assertFalse(actual.next());
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SQLRouteTest.java b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SQLRouteTest.java
index eb1962109a2..8d4a7711454 100644
--- a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SQLRouteTest.java
+++ b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SQLRouteTest.java
@@ -39,7 +39,7 @@ class SQLRouteTest {
ShardingRouteAssert.assertRoute(sql, params);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SubqueryRouteTest.java b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SubqueryRouteTest.java
index e3acc369358..0996fff984c 100644
--- a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SubqueryRouteTest.java
+++ b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/route/engine/type/standard/SubqueryRouteTest.java
@@ -51,7 +51,7 @@ class SubqueryRouteTest {
hintManager.close();
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/infra/exception/src/test/java/org/apache/shardingsphere/infra/exception/external/sql/ShardingSphereSQLExceptionTest.java b/infra/exception/src/test/java/org/apache/shardingsphere/infra/exception/external/sql/ShardingSphereSQLExceptionTest.java
index 6f9409779f1..3ce1fe3a016 100644
--- a/infra/exception/src/test/java/org/apache/shardingsphere/infra/exception/external/sql/ShardingSphereSQLExceptionTest.java
+++ b/infra/exception/src/test/java/org/apache/shardingsphere/infra/exception/external/sql/ShardingSphereSQLExceptionTest.java
@@ -29,7 +29,7 @@ class ShardingSphereSQLExceptionTest {
assertNotNull(new MalformedFormatMessageSQLException().getMessage());
}
- private static class MalformedFormatMessageSQLException extends ShardingSphereSQLException {
+ private static final class MalformedFormatMessageSQLException extends ShardingSphereSQLException {
private static final long serialVersionUID = 139616805450096292L;
diff --git a/infra/url/core/src/test/java/org/apache/shardingsphere/infra/url/core/ShardingSphereURLTest.java b/infra/url/core/src/test/java/org/apache/shardingsphere/infra/url/core/ShardingSphereURLTest.java
index 977864918af..4a8a4e42dd6 100644
--- a/infra/url/core/src/test/java/org/apache/shardingsphere/infra/url/core/ShardingSphereURLTest.java
+++ b/infra/url/core/src/test/java/org/apache/shardingsphere/infra/url/core/ShardingSphereURLTest.java
@@ -43,7 +43,7 @@ class ShardingSphereURLTest {
assertThat(actual.getQueryProps(), is(expectedProps));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/kernel/data-pipeline/api/src/main/java/org/apache/shardingsphere/data/pipeline/api/type/ShardingSpherePipelineDataSourceConfiguration.java b/kernel/data-pipeline/api/src/main/java/org/apache/shardingsphere/data/pipeline/api/type/ShardingSpherePipelineDataSourceConfiguration.java
index 78118ac0756..4a6a8591e27 100644
--- a/kernel/data-pipeline/api/src/main/java/org/apache/shardingsphere/data/pipeline/api/type/ShardingSpherePipelineDataSourceConfiguration.java
+++ b/kernel/data-pipeline/api/src/main/java/org/apache/shardingsphere/data/pipeline/api/type/ShardingSpherePipelineDataSourceConfiguration.java
@@ -138,7 +138,7 @@ public final class ShardingSpherePipelineDataSourceConfiguration implements Pipe
*/
@Getter
@Setter
- private static class YamlParameterConfiguration implements YamlConfiguration {
+ private static final class YamlParameterConfiguration implements YamlConfiguration {
private String databaseName;
diff --git a/kernel/data-pipeline/core/src/main/java/org/apache/shardingsphere/data/pipeline/core/util/PipelineDistributedBarrier.java b/kernel/data-pipeline/core/src/main/java/org/apache/shardingsphere/data/pipeline/core/util/PipelineDistributedBarrier.java
index 01f4946de2b..bd20f27e6c4 100644
--- a/kernel/data-pipeline/core/src/main/java/org/apache/shardingsphere/data/pipeline/core/util/PipelineDistributedBarrier.java
+++ b/kernel/data-pipeline/core/src/main/java/org/apache/shardingsphere/data/pipeline/core/util/PipelineDistributedBarrier.java
@@ -151,7 +151,7 @@ public final class PipelineDistributedBarrier {
@RequiredArgsConstructor
@Getter
- private static class InnerCountDownLatchHolder {
+ private static final class InnerCountDownLatchHolder {
private final int totalCount;
diff --git a/kernel/data-pipeline/core/src/test/java/org/apache/shardingsphere/data/pipeline/core/execute/AbstractPipelineLifecycleRunnableTest.java b/kernel/data-pipeline/core/src/test/java/org/apache/shardingsphere/data/pipeline/core/execute/AbstractPipelineLifecycleRunnableTest.java
index c4e0058fd9b..329f85968f9 100644
--- a/kernel/data-pipeline/core/src/test/java/org/apache/shardingsphere/data/pipeline/core/execute/AbstractPipelineLifecycleRunnableTest.java
+++ b/kernel/data-pipeline/core/src/test/java/org/apache/shardingsphere/data/pipeline/core/execute/AbstractPipelineLifecycleRunnableTest.java
@@ -72,7 +72,7 @@ class AbstractPipelineLifecycleRunnableTest {
assertThat(executor.runBlockingCount.get(), is(0));
}
- private static class FixturePipelineLifecycleRunnable extends AbstractPipelineLifecycleRunnable {
+ private static final class FixturePipelineLifecycleRunnable extends AbstractPipelineLifecycleRunnable {
private final AtomicInteger runBlockingCount = new AtomicInteger();
diff --git a/kernel/global-clock/distsql/handler/src/test/java/org/apache/shardingsphere/globalclock/distsql/handler/update/AlterGlobalClockRuleExecutorTest.java b/kernel/global-clock/distsql/handler/src/test/java/org/apache/shardingsphere/globalclock/distsql/handler/update/AlterGlobalClockRuleExecutorTest.java
index b63fba8c494..e931a171ef7 100644
--- a/kernel/global-clock/distsql/handler/src/test/java/org/apache/shardingsphere/globalclock/distsql/handler/update/AlterGlobalClockRuleExecutorTest.java
+++ b/kernel/global-clock/distsql/handler/src/test/java/org/apache/shardingsphere/globalclock/distsql/handler/update/AlterGlobalClockRuleExecutorTest.java
@@ -49,7 +49,7 @@ class AlterGlobalClockRuleExecutorTest extends GlobalRuleDefinitionExecutorTest
assertExecuteUpdate(ruleConfig, sqlStatement, matchedRuleConfig, null);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/kernel/sql-federation/compiler/src/test/java/org/apache/shardingsphere/sqlfederation/compiler/compiler/it/SQLStatementCompilerIT.java b/kernel/sql-federation/compiler/src/test/java/org/apache/shardingsphere/sqlfederation/compiler/compiler/it/SQLStatementCompilerIT.java
index f1369cc38d0..9ae12034bc8 100644
--- a/kernel/sql-federation/compiler/src/test/java/org/apache/shardingsphere/sqlfederation/compiler/compiler/it/SQLStatementCompilerIT.java
+++ b/kernel/sql-federation/compiler/src/test/java/org/apache/shardingsphere/sqlfederation/compiler/compiler/it/SQLStatementCompilerIT.java
@@ -249,7 +249,7 @@ class SQLStatementCompilerIT {
assertThat(actual, is(testcase.getAssertion().iterator().next().getExpectedResult()));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) throws IOException {
diff --git a/kernel/sql-federation/distsql/handler/src/test/java/org/apache/shardingsphere/sqlfederation/distsql/handler/update/AlterSQLFederationRuleExecutorTest.java b/kernel/sql-federation/distsql/handler/src/test/java/org/apache/shardingsphere/sqlfederation/distsql/handler/update/AlterSQLFederationRuleExecutorTest.java
index 42946a22b5c..8ac10e58488 100644
--- a/kernel/sql-federation/distsql/handler/src/test/java/org/apache/shardingsphere/sqlfederation/distsql/handler/update/AlterSQLFederationRuleExecutorTest.java
+++ b/kernel/sql-federation/distsql/handler/src/test/java/org/apache/shardingsphere/sqlfederation/distsql/handler/update/AlterSQLFederationRuleExecutorTest.java
@@ -52,7 +52,7 @@ class AlterSQLFederationRuleExecutorTest extends GlobalRuleDefinitionExecutorTes
assertExecuteUpdate(ruleConfig, sqlStatement, matchedRuleConfig, expectedException);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/kernel/sql-parser/distsql/handler/src/test/java/org/apache/shardingsphere/parser/distsql/handler/update/AlterSQLParserRuleExecutorTest.java b/kernel/sql-parser/distsql/handler/src/test/java/org/apache/shardingsphere/parser/distsql/handler/update/AlterSQLParserRuleExecutorTest.java
index a2d91405937..02e719c82ba 100644
--- a/kernel/sql-parser/distsql/handler/src/test/java/org/apache/shardingsphere/parser/distsql/handler/update/AlterSQLParserRuleExecutorTest.java
+++ b/kernel/sql-parser/distsql/handler/src/test/java/org/apache/shardingsphere/parser/distsql/handler/update/AlterSQLParserRuleExecutorTest.java
@@ -51,7 +51,7 @@ class AlterSQLParserRuleExecutorTest extends GlobalRuleDefinitionExecutorTest {
assertExecuteUpdate(ruleConfig, sqlStatement, matchedRuleConfig, null);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/kernel/sql-translator/distsql/handler/src/test/java/org/apache/shardingsphere/sqltranslator/distsql/handler/update/AlterSQLTranslatorRuleExecutorTest.java b/kernel/sql-translator/distsql/handler/src/test/java/org/apache/shardingsphere/sqltranslator/distsql/handler/update/AlterSQLTranslatorRuleExecutorTest.java
index b84f9cf1556..34860a1bb51 100644
--- a/kernel/sql-translator/distsql/handler/src/test/java/org/apache/shardingsphere/sqltranslator/distsql/handler/update/AlterSQLTranslatorRuleExecutorTest.java
+++ b/kernel/sql-translator/distsql/handler/src/test/java/org/apache/shardingsphere/sqltranslator/distsql/handler/update/AlterSQLTranslatorRuleExecutorTest.java
@@ -52,7 +52,7 @@ class AlterSQLTranslatorRuleExecutorTest extends GlobalRuleDefinitionExecutorTes
assertExecuteUpdate(ruleConfig, sqlStatement, matchedRuleConfig, null);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/kernel/transaction/distsql/handler/src/test/java/org/apache/shardingsphere/transaction/distsql/handler/update/AlterTransactionRuleExecutorTest.java b/kernel/transaction/distsql/handler/src/test/java/org/apache/shardingsphere/transaction/distsql/handler/update/AlterTransactionRuleExecutorTest.java
index c0e80d876de..30db5ca585e 100644
--- a/kernel/transaction/distsql/handler/src/test/java/org/apache/shardingsphere/transaction/distsql/handler/update/AlterTransactionRuleExecutorTest.java
+++ b/kernel/transaction/distsql/handler/src/test/java/org/apache/shardingsphere/transaction/distsql/handler/update/AlterTransactionRuleExecutorTest.java
@@ -54,7 +54,7 @@ class AlterTransactionRuleExecutorTest extends GlobalRuleDefinitionExecutorTest
assertExecuteUpdate(ruleConfig, sqlStatement, matchedRuleConfig, expectedException);
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/mode/node/src/test/java/org/apache/shardingsphere/mode/node/path/engine/generator/NodePathSegmentTest.java b/mode/node/src/test/java/org/apache/shardingsphere/mode/node/path/engine/generator/NodePathSegmentTest.java
index 99d676832c3..a97d7f7f00b 100644
--- a/mode/node/src/test/java/org/apache/shardingsphere/mode/node/path/engine/generator/NodePathSegmentTest.java
+++ b/mode/node/src/test/java/org/apache/shardingsphere/mode/node/path/engine/generator/NodePathSegmentTest.java
@@ -49,7 +49,7 @@ class NodePathSegmentTest {
@NodePathEntity("/foo/${fooVariable}")
@RequiredArgsConstructor
@Getter
- private static class NodePathFixture implements NodePath {
+ private static final class NodePathFixture implements NodePath {
private final String fooVariable;
}
diff --git a/parser/sql/statement/core/src/test/java/org/apache/shardingsphere/sql/parser/statement/core/extractor/ColumnExtractorParameterizedTest.java b/parser/sql/statement/core/src/test/java/org/apache/shardingsphere/sql/parser/statement/core/extractor/ColumnExtractorParameterizedTest.java
index 293455d7de4..41b3d7f7f6f 100644
--- a/parser/sql/statement/core/src/test/java/org/apache/shardingsphere/sql/parser/statement/core/extractor/ColumnExtractorParameterizedTest.java
+++ b/parser/sql/statement/core/src/test/java/org/apache/shardingsphere/sql/parser/statement/core/extractor/ColumnExtractorParameterizedTest.java
@@ -48,7 +48,7 @@ class ColumnExtractorParameterizedTest {
assertThat(columnSegments.iterator().next(), is(COLUMN_SEGMENT));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/session/ServerPreparedStatementRegistryTest.java b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/session/ServerPreparedStatementRegistryTest.java
index f2c4329e33e..1e33a956dec 100644
--- a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/session/ServerPreparedStatementRegistryTest.java
+++ b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/session/ServerPreparedStatementRegistryTest.java
@@ -45,7 +45,7 @@ class ServerPreparedStatementRegistryTest {
assertNull(registry.getPreparedStatement(1));
}
- private static class DummyServerPreparedStatement implements ServerPreparedStatement {
+ private static final class DummyServerPreparedStatement implements ServerPreparedStatement {
@Override
public String getSql() {
diff --git a/proxy/frontend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/frontend/firebird/authentication/authenticator/FirebirdAuthenticatorTypeTest.java b/proxy/frontend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/frontend/firebird/authentication/authenticator/FirebirdAuthenticatorTypeTest.java
index c5dd53a87e3..8dfe0a97517 100644
--- a/proxy/frontend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/frontend/firebird/authentication/authenticator/FirebirdAuthenticatorTypeTest.java
+++ b/proxy/frontend/dialect/firebird/src/test/java/org/apache/shardingsphere/proxy/frontend/firebird/authentication/authenticator/FirebirdAuthenticatorTypeTest.java
@@ -55,7 +55,7 @@ class FirebirdAuthenticatorTypeTest {
assertThat(actual.getClass().getSimpleName(), is(expectedAuthenticatorClassName));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/proxy/frontend/dialect/mysql/src/test/java/org/apache/shardingsphere/proxy/frontend/mysql/authentication/authenticator/MySQLAuthenticatorTypeTest.java b/proxy/frontend/dialect/mysql/src/test/java/org/apache/shardingsphere/proxy/frontend/mysql/authentication/authenticator/MySQLAuthenticatorTypeTest.java
index 5cdd4bf0a60..3bcd938ba4b 100644
--- a/proxy/frontend/dialect/mysql/src/test/java/org/apache/shardingsphere/proxy/frontend/mysql/authentication/authenticator/MySQLAuthenticatorTypeTest.java
+++ b/proxy/frontend/dialect/mysql/src/test/java/org/apache/shardingsphere/proxy/frontend/mysql/authentication/authenticator/MySQLAuthenticatorTypeTest.java
@@ -48,7 +48,7 @@ class MySQLAuthenticatorTypeTest {
assertThat(actual.getAuthenticationMethodName(), is(expectedAuthenticatorMethodName));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/proxy/frontend/dialect/opengauss/src/test/java/org/apache/shardingsphere/proxy/frontend/opengauss/authentication/authenticator/OpenGaussAuthenticatorTypeTest.java b/proxy/frontend/dialect/opengauss/src/test/java/org/apache/shardingsphere/proxy/frontend/opengauss/authentication/authenticator/OpenGaussAuthenticatorTypeTest.java
index fd6b06d4b34..abe540edbe6 100644
--- a/proxy/frontend/dialect/opengauss/src/test/java/org/apache/shardingsphere/proxy/frontend/opengauss/authentication/authenticator/OpenGaussAuthenticatorTypeTest.java
+++ b/proxy/frontend/dialect/opengauss/src/test/java/org/apache/shardingsphere/proxy/frontend/opengauss/authentication/authenticator/OpenGaussAuthenticatorTypeTest.java
@@ -48,7 +48,7 @@ class OpenGaussAuthenticatorTypeTest {
assertThat(actual.getAuthenticationMethodName(), is(expectedAuthenticatorMethodName));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/proxy/frontend/dialect/postgresql/src/test/java/org/apache/shardingsphere/proxy/frontend/postgresql/authentication/authenticator/PostgreSQLAuthenticatorTypeTest.java b/proxy/frontend/dialect/postgresql/src/test/java/org/apache/shardingsphere/proxy/frontend/postgresql/authentication/authenticator/PostgreSQLAuthenticatorTypeTest.java
index baf619d6b97..aa9e62af289 100644
--- a/proxy/frontend/dialect/postgresql/src/test/java/org/apache/shardingsphere/proxy/frontend/postgresql/authentication/authenticator/PostgreSQLAuthenticatorTypeTest.java
+++ b/proxy/frontend/dialect/postgresql/src/test/java/org/apache/shardingsphere/proxy/frontend/postgresql/authentication/authenticator/PostgreSQLAuthenticatorTypeTest.java
@@ -48,7 +48,7 @@ class PostgreSQLAuthenticatorTypeTest {
assertThat(actual.getAuthenticationMethodName(), is(expectedAuthenticatorMethodName));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/alterresource/SwitchingTransactionRuleTestCase.java b/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/alterresource/SwitchingTransactionRuleTestCase.java
index 31f491bf4c7..a80b17a85aa 100644
--- a/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/alterresource/SwitchingTransactionRuleTestCase.java
+++ b/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/alterresource/SwitchingTransactionRuleTestCase.java
@@ -94,7 +94,7 @@ public final class SwitchingTransactionRuleTestCase extends BaseTransactionTestC
@RequiredArgsConstructor
@Getter
- private static class AlterTransactionRuleTask implements Runnable {
+ private static final class AlterTransactionRuleTask implements Runnable {
private final TransactionContainerComposer containerComposer;
@@ -181,7 +181,7 @@ public final class SwitchingTransactionRuleTestCase extends BaseTransactionTestC
@RequiredArgsConstructor
@Getter
- private static class TransactionOperationsTask implements Runnable {
+ private static final class TransactionOperationsTask implements Runnable {
private static final AtomicInteger ID_COUNTER = new AtomicInteger();
diff --git a/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/classictransfer/ClassicTransferTestCase.java b/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/classictransfer/ClassicTransferTestCase.java
index d3df65119ee..004c5880bf1 100644
--- a/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/classictransfer/ClassicTransferTestCase.java
+++ b/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/operation/transaction/cases/classictransfer/ClassicTransferTestCase.java
@@ -90,7 +90,7 @@ public final class ClassicTransferTestCase extends BaseTransactionTestCase {
@RequiredArgsConstructor
@Getter
- private static class UpdateAccountTask implements Runnable {
+ private static final class UpdateAccountTask implements Runnable {
private final DataSource dataSource;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/sql/env/DataSetEnvironmentManager.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/sql/env/DataSetEnvironmentManager.java
index 671b6df6f5d..56d0f7ab55d 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/sql/env/DataSetEnvironmentManager.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/sql/env/DataSetEnvironmentManager.java
@@ -184,7 +184,7 @@ public final class DataSetEnvironmentManager {
}
@RequiredArgsConstructor
- private static class InsertTask implements Callable<Void> {
+ private static final class InsertTask implements Callable<Void> {
private final DataSource dataSource;
@@ -214,7 +214,7 @@ public final class DataSetEnvironmentManager {
}
@RequiredArgsConstructor
- private static class DeleteTask implements Callable<Void> {
+ private static final class DeleteTask implements Callable<Void> {
private final DataSource dataSource;
diff --git a/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereArgumentVerifyMatchers.java b/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereArgumentVerifyMatchers.java
index 7ca5aba4a9a..e8cd6893cb6 100644
--- a/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereArgumentVerifyMatchers.java
+++ b/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereArgumentVerifyMatchers.java
@@ -47,7 +47,7 @@ public final class ShardingSphereArgumentVerifyMatchers {
}
@RequiredArgsConstructor
- private static class DeepEqualsMatcher implements ArgumentMatcher<Object> {
+ private static final class DeepEqualsMatcher implements ArgumentMatcher<Object> {
private final Object wanted;
diff --git a/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereAssertionMatchers.java b/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereAssertionMatchers.java
index 25f9d4090bb..fe63da1d6d5 100644
--- a/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereAssertionMatchers.java
+++ b/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/matcher/ShardingSphereAssertionMatchers.java
@@ -43,7 +43,7 @@ public final class ShardingSphereAssertionMatchers {
}
@RequiredArgsConstructor
- private static class DeepEqualMatcher<T> extends BaseMatcher<T> {
+ private static final class DeepEqualMatcher<T> extends BaseMatcher<T> {
private final T expectedValue;
diff --git a/test/it/binder/src/test/java/org/apache/shardingsphere/test/it/sql/binder/SQLBinderIT.java b/test/it/binder/src/test/java/org/apache/shardingsphere/test/it/sql/binder/SQLBinderIT.java
index 8d3590d6b61..75d24cf7a62 100644
--- a/test/it/binder/src/test/java/org/apache/shardingsphere/test/it/sql/binder/SQLBinderIT.java
+++ b/test/it/binder/src/test/java/org/apache/shardingsphere/test/it/sql/binder/SQLBinderIT.java
@@ -171,7 +171,7 @@ public abstract class SQLBinderIT {
return result;
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimizer/sqlnode/converter/SQLNodeConverterEngineIT.java b/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimizer/sqlnode/converter/SQLNodeConverterEngineIT.java
index 8184f0c721d..eff3740c3e5 100644
--- a/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimizer/sqlnode/converter/SQLNodeConverterEngineIT.java
+++ b/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimizer/sqlnode/converter/SQLNodeConverterEngineIT.java
@@ -77,7 +77,7 @@ public abstract class SQLNodeConverterEngineIT {
return new SQLStatementVisitorEngine(databaseType).visit(new SQLParserEngine(databaseType, new CacheOption(128, 1024L)).parse(sql, false));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/ExternalSQLParserIT.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/ExternalSQLParserIT.java
index 56089ab8c42..f3597d7c553 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/ExternalSQLParserIT.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/ExternalSQLParserIT.java
@@ -71,7 +71,7 @@ public abstract class ExternalSQLParserIT {
return SQLParserExternalITEnvironment.getInstance().isSqlParserITEnabled();
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) throws ReflectiveOperationException {
diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalSQLParserIT.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalSQLParserIT.java
index 974147878ef..1c70baed83e 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalSQLParserIT.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalSQLParserIT.java
@@ -65,7 +65,7 @@ public abstract class InternalSQLParserIT {
: new SQLStatementVisitorEngine(databaseType).visit(new SQLParserEngine(databaseType, new CacheOption(128, 1024L)).parse(sql, false));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalUnsupportedSQLParserIT.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalUnsupportedSQLParserIT.java
index 92b727374fd..2ffc237cd8c 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalUnsupportedSQLParserIT.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalUnsupportedSQLParserIT.java
@@ -51,7 +51,7 @@ public abstract class InternalUnsupportedSQLParserIT {
assertThrows(SQLParsingException.class, () -> new SQLParserEngine("H2".equals(databaseType) ? "MySQL" : databaseType, cacheOption).parse(sql, false));
}
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {
diff --git a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewriter/engine/SQLRewriterIT.java b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewriter/engine/SQLRewriterIT.java
index 6fc36d7c35a..3152034ba0d 100644
--- a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewriter/engine/SQLRewriterIT.java
+++ b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewriter/engine/SQLRewriterIT.java
@@ -165,7 +165,7 @@ public abstract class SQLRewriterIT {
protected abstract void mockDatabaseRules(Collection<ShardingSphereRule> rules, String schemaName, SQLStatement sqlStatement);
- private static class TestCaseArgumentsProvider implements ArgumentsProvider {
+ private static final class TestCaseArgumentsProvider implements ArgumentsProvider {
@Override
public Stream<? extends Arguments> provideArguments(final ParameterDeclarations parameters, final ExtensionContext context) {