You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicecomb.apache.org by li...@apache.org on 2022/05/12 10:30:06 UTC

[servicecomb-java-chassis] branch master updated: [#SCB-2504] remove unused "LOGGER" private field (#2879)

This is an automated email from the ASF dual-hosted git repository.

liubao 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 d36bd8848 [#SCB-2504] remove unused "LOGGER" private field (#2879)
d36bd8848 is described below

commit d36bd8848485a085d01a040c650a591854738c4d
Author: david6969xin <86...@users.noreply.github.com>
AuthorDate: Thu May 12 18:29:59 2022 +0800

    [#SCB-2504] remove unused "LOGGER" private field (#2879)
---
 .../main/java/org/apache/servicecomb/common/rest/UploadConfig.java    | 4 ----
 .../java/org/apache/servicecomb/core/transport/AbstractTransport.java | 3 ---
 .../src/main/java/org/apache/servicecomb/config/YAMLUtil.java         | 4 ----
 .../apache/servicecomb/metrics/core/meter/os/cpu/ProcessCpuUsage.java | 4 ----
 .../apache/servicecomb/swagger/engine/SwaggerProducerOperation.java   | 3 ---
 5 files changed, 18 deletions(-)

diff --git a/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/UploadConfig.java b/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/UploadConfig.java
index 40d3e63d5..302de2c8b 100644
--- a/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/UploadConfig.java
+++ b/common/common-rest/src/main/java/org/apache/servicecomb/common/rest/UploadConfig.java
@@ -18,13 +18,9 @@ package org.apache.servicecomb.common.rest;
 
 import javax.servlet.MultipartConfigElement;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.netflix.config.DynamicPropertyFactory;
 
 public class UploadConfig {
-  private static final Logger LOGGER = LoggerFactory.getLogger(UploadConfig.class);
 
   /**
    * null means not support upload
diff --git a/core/src/main/java/org/apache/servicecomb/core/transport/AbstractTransport.java b/core/src/main/java/org/apache/servicecomb/core/transport/AbstractTransport.java
index 4013c2989..c75ce25f8 100644
--- a/core/src/main/java/org/apache/servicecomb/core/transport/AbstractTransport.java
+++ b/core/src/main/java/org/apache/servicecomb/core/transport/AbstractTransport.java
@@ -29,13 +29,10 @@ import org.apache.servicecomb.foundation.common.net.NetUtils;
 import org.apache.servicecomb.foundation.common.net.URIEndpointObject;
 import org.apache.servicecomb.foundation.vertx.SharedVertxFactory;
 import org.apache.servicecomb.registry.RegistrationManager;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import io.vertx.core.Vertx;
 
 public abstract class AbstractTransport implements Transport {
-  private static final Logger LOGGER = LoggerFactory.getLogger(AbstractTransport.class);
 
   /*
    * 用于参数传递:比如向RestServerVerticle传递endpoint地址。
diff --git a/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/YAMLUtil.java b/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/YAMLUtil.java
index b5c793e14..ac3b00a9c 100644
--- a/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/YAMLUtil.java
+++ b/foundations/foundation-config/src/main/java/org/apache/servicecomb/config/YAMLUtil.java
@@ -24,8 +24,6 @@ import java.io.InputStream;
 import java.util.LinkedHashMap;
 import java.util.Map;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import org.yaml.snakeyaml.TypeDescription;
 import org.yaml.snakeyaml.Yaml;
 import org.yaml.snakeyaml.constructor.Constructor;
@@ -37,8 +35,6 @@ import org.yaml.snakeyaml.constructor.SafeConstructor;
 public final class YAMLUtil {
   private static final Yaml SAFE_PARSER = new Yaml(new SafeConstructor());
 
-  private static final Logger LOGGER = LoggerFactory.getLogger(YAMLUtil.class);
-
   private YAMLUtil() {
   }
 
diff --git a/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/os/cpu/ProcessCpuUsage.java b/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/os/cpu/ProcessCpuUsage.java
index f5213c82e..00ec8cb5e 100644
--- a/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/os/cpu/ProcessCpuUsage.java
+++ b/metrics/metrics-core/src/main/java/org/apache/servicecomb/metrics/core/meter/os/cpu/ProcessCpuUsage.java
@@ -18,9 +18,6 @@ package org.apache.servicecomb.metrics.core.meter.os.cpu;
 
 import java.io.IOException;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.netflix.spectator.api.Id;
 
 /*
@@ -37,7 +34,6 @@ import com.netflix.spectator.api.Id;
  *
  */
 public class ProcessCpuUsage extends AbstractCpuUsage {
-  private static final Logger LOGGER = LoggerFactory.getLogger(ProcessCpuUsage.class);
 
   private final Period busy = new Period();
 
diff --git a/swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/engine/SwaggerProducerOperation.java b/swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/engine/SwaggerProducerOperation.java
index 5ded88938..d601536d8 100644
--- a/swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/engine/SwaggerProducerOperation.java
+++ b/swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/engine/SwaggerProducerOperation.java
@@ -26,11 +26,8 @@ import org.apache.servicecomb.swagger.generator.core.model.SwaggerOperation;
 import org.apache.servicecomb.swagger.invocation.arguments.producer.ProducerArgumentsMapper;
 import org.apache.servicecomb.swagger.invocation.extension.ProducerInvokeExtension;
 import org.apache.servicecomb.swagger.invocation.response.producer.ProducerResponseMapper;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class SwaggerProducerOperation {
-  private static final Logger LOGGER = LoggerFactory.getLogger(SwaggerProducerOperation.class);
 
   // 因为存在aop场景,所以,producerClass不一定等于producerInstance.getClass()
   private Class<?> producerClass;