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 <[email protected]>
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) {