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) {

Reply via email to