You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicecomb.apache.org by sh...@apache.org on 2022/06/10 23:26:34 UTC
[servicecomb-java-chassis] branch master updated: [SCB-2558] enable checkstyle ModifierOrder (#3055)
This is an automated email from the ASF dual-hosted git repository.
shoothzj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git
The following commit(s) were added to refs/heads/master by this push:
new 4ec960080 [SCB-2558] enable checkstyle ModifierOrder (#3055)
4ec960080 is described below
commit 4ec960080c7aebd3ec24616d82f649dfde81334a
Author: TingTing Wang <19...@qq.com>
AuthorDate: Sat Jun 11 07:26:28 2022 +0800
[SCB-2558] enable checkstyle ModifierOrder (#3055)
---
ci/checkstyle/checkstyle.xml | 1 +
core/src/main/java/org/apache/servicecomb/core/SCBEngine.java | 2 +-
.../servicecomb/demo/edge/service/CustomResponseMetaMapper.java | 2 +-
.../foundation/common/utils/AbstractRestObjectMapper.java | 2 +-
.../org/apache/servicecomb/foundation/common/utils/RSAUtils.java | 8 ++++----
.../schema/deserializer/scalar/AbstractScalarReadSchemas.java | 4 ++--
.../servicecomb/registry/config/AbstractPropertiesLoader.java | 4 ++--
.../servicecomb/governance/handler/AbstractGovernanceHandler.java | 6 +++---
.../servicecomb/loadbalance/ServiceCombLoadBalancerStats.java | 2 +-
.../apache/servicecomb/authentication/RSAAuthenticationToken.java | 2 +-
.../authentication/provider/RSAProviderTokenManager.java | 2 +-
.../apache/servicecomb/metrics/core/TestHealthBootListener.java | 4 ++--
.../apache/servicecomb/metrics/core/TestMetricsBootListener.java | 4 ++--
.../org/apache/servicecomb/serviceregistry/task/AbstractTask.java | 2 +-
.../client/http/TestServiceRegistryClientImpl.java | 2 +-
.../servicecomb/swagger/generator/core/schema/InvalidType.java | 2 +-
16 files changed, 25 insertions(+), 24 deletions(-)
diff --git a/ci/checkstyle/checkstyle.xml b/ci/checkstyle/checkstyle.xml
index 1d3953d9b..10a7490f8 100644
--- a/ci/checkstyle/checkstyle.xml
+++ b/ci/checkstyle/checkstyle.xml
@@ -34,6 +34,7 @@
<module name="TreeWalker">
<module name="AvoidStarImport"/>
<module name="EmptyStatement"/>
+ <module name="ModifierOrder"/>
<module name="NoWhitespaceAfter">
<!-- Checks that there is no whitespace after various unary operators.
Linebreaks are allowed.
diff --git a/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java b/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java
index 6222199eb..9ed9daa1c 100644
--- a/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java
+++ b/core/src/main/java/org/apache/servicecomb/core/SCBEngine.java
@@ -87,7 +87,7 @@ public class SCBEngine {
private static final Object initializationLock = new Object();
- private volatile static SCBEngine INSTANCE;
+ private static volatile SCBEngine INSTANCE;
private ApplicationContext applicationContext;
diff --git a/demo/demo-edge/edge-service/src/main/java/org/apache/servicecomb/demo/edge/service/CustomResponseMetaMapper.java b/demo/demo-edge/edge-service/src/main/java/org/apache/servicecomb/demo/edge/service/CustomResponseMetaMapper.java
index eca615829..7aa294c83 100644
--- a/demo/demo-edge/edge-service/src/main/java/org/apache/servicecomb/demo/edge/service/CustomResponseMetaMapper.java
+++ b/demo/demo-edge/edge-service/src/main/java/org/apache/servicecomb/demo/edge/service/CustomResponseMetaMapper.java
@@ -26,7 +26,7 @@ import com.fasterxml.jackson.databind.JavaType;
import com.fasterxml.jackson.databind.type.SimpleType;
public class CustomResponseMetaMapper implements ResponseMetaMapper {
- private final static Map<Integer, JavaType> CODES = new HashMap<>(1);
+ private static final Map<Integer, JavaType> CODES = new HashMap<>(1);
static {
CODES.put(500, SimpleType.constructUnsafe(IllegalStateErrorData.class));
diff --git a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/AbstractRestObjectMapper.java b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/AbstractRestObjectMapper.java
index 43f106270..5069e6a39 100644
--- a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/AbstractRestObjectMapper.java
+++ b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/AbstractRestObjectMapper.java
@@ -30,5 +30,5 @@ public abstract class AbstractRestObjectMapper extends ObjectMapper {
super(src);
}
- abstract public String convertToString(Object value) throws Exception;
+ public abstract String convertToString(Object value) throws Exception;
}
diff --git a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java
index 6922c763c..5923f9b5f 100644
--- a/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java
+++ b/foundations/foundation-common/src/main/java/org/apache/servicecomb/foundation/common/utils/RSAUtils.java
@@ -36,13 +36,13 @@ import org.slf4j.LoggerFactory;
public class RSAUtils {
- private final static Logger LOGGER = LoggerFactory.getLogger(RSAUtils.class);
+ private static final Logger LOGGER = LoggerFactory.getLogger(RSAUtils.class);
- private final static String RSA_ALG = "RSA";
+ private static final String RSA_ALG = "RSA";
- private final static String SIGN_ALG = "SHA256withRSA";
+ private static final String SIGN_ALG = "SHA256withRSA";
- private final static int KEY_SIZE = 2048;
+ private static final int KEY_SIZE = 2048;
private static final Base64.Encoder encoder = Base64.getEncoder();
diff --git a/foundations/foundation-protobuf/src/main/java/org/apache/servicecomb/foundation/protobuf/internal/schema/deserializer/scalar/AbstractScalarReadSchemas.java b/foundations/foundation-protobuf/src/main/java/org/apache/servicecomb/foundation/protobuf/internal/schema/deserializer/scalar/AbstractScalarReadSchemas.java
index a1f8e06bb..345fd477d 100644
--- a/foundations/foundation-protobuf/src/main/java/org/apache/servicecomb/foundation/protobuf/internal/schema/deserializer/scalar/AbstractScalarReadSchemas.java
+++ b/foundations/foundation-protobuf/src/main/java/org/apache/servicecomb/foundation/protobuf/internal/schema/deserializer/scalar/AbstractScalarReadSchemas.java
@@ -23,7 +23,7 @@ import io.protostuff.compiler.model.Field;
import io.protostuff.runtime.FieldSchema;
public class AbstractScalarReadSchemas {
- static abstract class AbstractIntSchema<T> extends FieldSchema<T> {
+ abstract static class AbstractIntSchema<T> extends FieldSchema<T> {
protected final Setter<T, Object> setter;
public AbstractIntSchema(Field protoField, PropertyDescriptor propertyDescriptor) {
@@ -32,7 +32,7 @@ public class AbstractScalarReadSchemas {
}
}
- static abstract class AbstractLongSchema<T> extends FieldSchema<T> {
+ abstract static class AbstractLongSchema<T> extends FieldSchema<T> {
protected final Setter<T, Object> setter;
public AbstractLongSchema(Field protoField, PropertyDescriptor propertyDescriptor) {
diff --git a/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/config/AbstractPropertiesLoader.java b/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/config/AbstractPropertiesLoader.java
index 1944063d2..723ec0a5b 100644
--- a/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/config/AbstractPropertiesLoader.java
+++ b/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/config/AbstractPropertiesLoader.java
@@ -40,9 +40,9 @@ public abstract class AbstractPropertiesLoader {
return propertiesMap;
}
- abstract protected Map<String, String> readProperties(Configuration configuration);
+ protected abstract Map<String, String> readProperties(Configuration configuration);
- abstract protected String readPropertiesExtendedClass(Configuration configuration);
+ protected abstract String readPropertiesExtendedClass(Configuration configuration);
private void loadPropertiesFromConfigMap(Configuration configuration, Map<String, String> propertiesMap) {
propertiesMap.putAll(readProperties(configuration));
diff --git a/governance/src/main/java/org/apache/servicecomb/governance/handler/AbstractGovernanceHandler.java b/governance/src/main/java/org/apache/servicecomb/governance/handler/AbstractGovernanceHandler.java
index 4f5e0279a..9e4c02ddf 100644
--- a/governance/src/main/java/org/apache/servicecomb/governance/handler/AbstractGovernanceHandler.java
+++ b/governance/src/main/java/org/apache/servicecomb/governance/handler/AbstractGovernanceHandler.java
@@ -65,11 +65,11 @@ public abstract class AbstractGovernanceHandler<PROCESSOR, POLICY extends Abstra
return processor;
}
- abstract protected String createKey(GovernanceRequest governanceRequest, POLICY policy);
+ protected abstract String createKey(GovernanceRequest governanceRequest, POLICY policy);
- abstract protected POLICY matchPolicy(GovernanceRequest governanceRequest);
+ protected abstract POLICY matchPolicy(GovernanceRequest governanceRequest);
- abstract protected PROCESSOR createProcessor(GovernanceRequest governanceRequest, POLICY policy);
+ protected abstract PROCESSOR createProcessor(GovernanceRequest governanceRequest, POLICY policy);
protected void onConfigurationChanged(String key) {
processors.remove(key);
diff --git a/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/ServiceCombLoadBalancerStats.java b/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/ServiceCombLoadBalancerStats.java
index 553746013..fac9ba177 100644
--- a/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/ServiceCombLoadBalancerStats.java
+++ b/handlers/handler-loadbalance/src/main/java/org/apache/servicecomb/loadbalance/ServiceCombLoadBalancerStats.java
@@ -41,7 +41,7 @@ import com.netflix.config.DynamicPropertyFactory;
* Add special stats that com.netflix.loadbalancer.LoadBalancerStats not provided
*/
public class ServiceCombLoadBalancerStats {
- private final static Logger LOGGER = LoggerFactory.getLogger(ServiceCombLoadBalancerStats.class);
+ private static final Logger LOGGER = LoggerFactory.getLogger(ServiceCombLoadBalancerStats.class);
private final Map<ServiceCombServer, ServiceCombServerStats> pingView = new ConcurrentHashMap<>();
diff --git a/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/RSAAuthenticationToken.java b/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/RSAAuthenticationToken.java
index 01db32f9a..935114909 100644
--- a/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/RSAAuthenticationToken.java
+++ b/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/RSAAuthenticationToken.java
@@ -24,7 +24,7 @@ package org.apache.servicecomb.authentication;
*/
public class RSAAuthenticationToken {
- public final static long TOKEN_ACTIVE_TIME = 24 * 60 * 60 * 1000;
+ public static final long TOKEN_ACTIVE_TIME = 24 * 60 * 60 * 1000;
private final String instanceId;
diff --git a/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/provider/RSAProviderTokenManager.java b/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/provider/RSAProviderTokenManager.java
index bfb9edef7..76b2e3bf9 100644
--- a/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/provider/RSAProviderTokenManager.java
+++ b/handlers/handler-publickey-auth/src/main/java/org/apache/servicecomb/authentication/provider/RSAProviderTokenManager.java
@@ -36,7 +36,7 @@ import com.google.common.cache.CacheBuilder;
public class RSAProviderTokenManager {
- private final static Logger LOGGER = LoggerFactory.getLogger(RSAProviderTokenManager.class);
+ private static final Logger LOGGER = LoggerFactory.getLogger(RSAProviderTokenManager.class);
private final Cache<RSAAuthenticationToken, Boolean> validatedToken = CacheBuilder.newBuilder()
.expireAfterAccess(getExpiredTime(), TimeUnit.MILLISECONDS)
diff --git a/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestHealthBootListener.java b/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestHealthBootListener.java
index 1c8624fc0..4e28ecaba 100644
--- a/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestHealthBootListener.java
+++ b/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestHealthBootListener.java
@@ -52,7 +52,7 @@ public class TestHealthBootListener {
final BootEvent event = new BootEvent();
final ProducerMeta producerMeta = new ProducerMeta();
final SCBEngine scbEngine = new SCBEngine() {
- final public ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
+ public final ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
@Override
public void addProducerMeta(String schemaId, Object instance) {
@@ -83,7 +83,7 @@ public class TestHealthBootListener {
final List<ProducerMeta> producerMetas = new ArrayList<>();
final BootEvent event = new BootEvent();
final SCBEngine scbEngine = new SCBEngine() {
- final public ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
+ public final ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
@Override
public void addProducerMeta(String schemaId, Object instance) {
diff --git a/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestMetricsBootListener.java b/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestMetricsBootListener.java
index 302f5cc59..4b462dd5e 100644
--- a/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestMetricsBootListener.java
+++ b/metrics/metrics-core/src/test/java/org/apache/servicecomb/metrics/core/TestMetricsBootListener.java
@@ -50,7 +50,7 @@ public class TestMetricsBootListener {
final BootEvent event = new BootEvent();
final ProducerMeta producerMeta = new ProducerMeta();
final SCBEngine scbEngine = new SCBEngine() {
- final public ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
+ public final ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
@Override
public void addProducerMeta(String schemaId, Object instance) {
@@ -81,7 +81,7 @@ public class TestMetricsBootListener {
final List<ProducerMeta> producerMetas = new ArrayList<>();
final BootEvent event = new BootEvent();
final SCBEngine scbEngine = new SCBEngine() {
- final public ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
+ public final ProducerProviderManager producerProviderManager = new ProducerProviderManager(this) {
@Override
public void addProducerMeta(String schemaId, Object instance) {
diff --git a/service-registry/registry-service-center/src/main/java/org/apache/servicecomb/serviceregistry/task/AbstractTask.java b/service-registry/registry-service-center/src/main/java/org/apache/servicecomb/serviceregistry/task/AbstractTask.java
index c9d89153e..f015cb367 100644
--- a/service-registry/registry-service-center/src/main/java/org/apache/servicecomb/serviceregistry/task/AbstractTask.java
+++ b/service-registry/registry-service-center/src/main/java/org/apache/servicecomb/serviceregistry/task/AbstractTask.java
@@ -59,7 +59,7 @@ public abstract class AbstractTask implements Runnable {
return this.taskStatus;
}
- abstract protected void doRun();
+ protected abstract void doRun();
protected boolean isSameMicroservice(Microservice otherMicroservice) {
return microservice.getServiceName().equals(otherMicroservice.getServiceName());
diff --git a/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/client/http/TestServiceRegistryClientImpl.java b/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/client/http/TestServiceRegistryClientImpl.java
index 71f796ee2..b6c1d75aa 100644
--- a/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/client/http/TestServiceRegistryClientImpl.java
+++ b/service-registry/registry-service-center/src/test/java/org/apache/servicecomb/serviceregistry/client/http/TestServiceRegistryClientImpl.java
@@ -148,7 +148,7 @@ public class TestServiceRegistryClientImpl {
ArchaiusUtils.resetConfig();
}
- static abstract class RegisterSchemaTester {
+ abstract static class RegisterSchemaTester {
void run() {
Logger rootLogger = Logger.getRootLogger();
diff --git a/swagger/swagger-generator/generator-core/src/test/java/org/apache/servicecomb/swagger/generator/core/schema/InvalidType.java b/swagger/swagger-generator/generator-core/src/test/java/org/apache/servicecomb/swagger/generator/core/schema/InvalidType.java
index b8d2223ba..9b4bf521a 100644
--- a/swagger/swagger-generator/generator-core/src/test/java/org/apache/servicecomb/swagger/generator/core/schema/InvalidType.java
+++ b/swagger/swagger-generator/generator-core/src/test/java/org/apache/servicecomb/swagger/generator/core/schema/InvalidType.java
@@ -25,7 +25,7 @@ public class InvalidType {
}
- public static abstract class InvalidClass {
+ public abstract static class InvalidClass {
}