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

albumenj pushed a commit to branch 3.2
in repository https://gitbox.apache.org/repos/asf/dubbo.git


The following commit(s) were added to refs/heads/3.2 by this push:
     new d8e6dfa42a fix: code format (#11258)
d8e6dfa42a is described below

commit d8e6dfa42a22e9c21697ad37709abec7ae33895b
Author: Xingpeng Li <[email protected]>
AuthorDate: Mon Jan 9 20:02:29 2023 +0800

    fix: code format (#11258)
    
    * fix: code format
    
    * fix: code format
    
    * fix: code format
---
 .../common/metrics/collector/DefaultMetricsCollector.java      |  6 +-----
 .../metrics/collector/stat/DefaultMetricsStatHandler.java      |  4 +---
 .../apache/dubbo/metrics/filter/MetricsCollectExecutor.java    | 10 +++++-----
 .../java/org/apache/dubbo/metrics/filter/MetricsFilter.java    |  2 +-
 4 files changed, 8 insertions(+), 14 deletions(-)

diff --git 
a/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/DefaultMetricsCollector.java
 
b/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/DefaultMetricsCollector.java
index c9e5bfeb38..1760ccb59f 100644
--- 
a/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/DefaultMetricsCollector.java
+++ 
b/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/DefaultMetricsCollector.java
@@ -140,9 +140,7 @@ public class DefaultMetricsCollector implements 
MetricsCollector {
             .ifPresent(map-> map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUESTS_SUCCEED, k.getTags(), REQUESTS, 
v::get))));
 
         doExecute(RequestEvent.Type.FAILED, 
MetricsStatHandler::get).filter(e->!e.isEmpty())
-            .ifPresent(map->{
-                map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUESTS_FAILED, k.getTags(), REQUESTS, 
v::get)));
-            });
+            .ifPresent(map-> map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUESTS_FAILED, k.getTags(), REQUESTS, 
v::get))));
 
         doExecute(RequestEvent.Type.PROCESSING, 
MetricsStatHandler::get).filter(e->!e.isEmpty())
             .ifPresent(map-> map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUESTS_PROCESSING, k.getTags(), REQUESTS, 
v::get))));
@@ -150,13 +148,11 @@ public class DefaultMetricsCollector implements 
MetricsCollector {
         doExecute(RequestEvent.Type.BUSINESS_FAILED, 
MetricsStatHandler::get).filter(e->!e.isEmpty())
             .ifPresent(map-> map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUEST_BUSINESS_FAILED, k.getTags(), 
REQUESTS, v::get))));
 
-
         doExecute(RequestEvent.Type.REQUEST_TIMEOUT, 
MetricsStatHandler::get).filter(e->!e.isEmpty())
             .ifPresent(map-> map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUESTS_TIMEOUT_AGG, k.getTags(), 
REQUESTS, v::get))));
 
         doExecute(RequestEvent.Type.REQUEST_LIMIT, 
MetricsStatHandler::get).filter(e->!e.isEmpty())
             .ifPresent(map-> map.forEach((k, v) -> list.add(new 
GaugeMetricSample(MetricsKey.METRIC_REQUESTS_LIMIT_AGG, k.getTags(), REQUESTS, 
v::get))));
-
     }
 
     private void collectRT(List<MetricSample> list) {
diff --git 
a/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/stat/DefaultMetricsStatHandler.java
 
b/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/stat/DefaultMetricsStatHandler.java
index 1670b9d36a..3d93374b25 100644
--- 
a/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/stat/DefaultMetricsStatHandler.java
+++ 
b/dubbo-common/src/main/java/org/apache/dubbo/common/metrics/collector/stat/DefaultMetricsStatHandler.java
@@ -53,7 +53,6 @@ public class DefaultMetricsStatHandler implements 
MetricsStatHandler {
         
this.doExecute(interfaceName,methodName,group,version,(metric,counts)->{
             AtomicLong count = counts.computeIfAbsent(metric, k -> new 
AtomicLong(0L));
             count.incrementAndGet();
-
         });
     }
 
@@ -69,6 +68,5 @@ public class DefaultMetricsStatHandler implements 
MetricsStatHandler {
         return counts;
     }
 
-    public  void doNotify(MethodMetric metric){}
-
+    public void doNotify(MethodMetric metric){}
 }
diff --git 
a/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsCollectExecutor.java
 
b/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsCollectExecutor.java
index 2930334ca5..e321b00818 100644
--- 
a/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsCollectExecutor.java
+++ 
b/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsCollectExecutor.java
@@ -31,11 +31,11 @@ import org.apache.dubbo.rpc.RpcInvocation;
 public class MetricsCollectExecutor {
 
     private final DefaultMetricsCollector collector;
-    private final Invocation    invocation;
-    private String  interfaceName;
-    private String  methodName;
-    private String  group;
-    private String  version;
+    private final Invocation invocation;
+    private String interfaceName;
+    private String methodName;
+    private String group;
+    private String version;
 
 
     public MetricsCollectExecutor(DefaultMetricsCollector collector, 
Invocation invocation) {
diff --git 
a/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsFilter.java
 
b/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsFilter.java
index f13c90d6a9..d0e4570968 100644
--- 
a/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsFilter.java
+++ 
b/dubbo-metrics/dubbo-metrics-api/src/main/java/org/apache/dubbo/metrics/filter/MetricsFilter.java
@@ -62,7 +62,7 @@ public class MetricsFilter implements Filter, 
BaseFilter.Listener, ScopeModelAwa
 
     @Override
     public void onError(Throwable t, Invoker<?> invoker, Invocation 
invocation) {
-        collect(invocation,collector-> collector.throwExecute(t));
+        collect(invocation, collector-> collector.throwExecute(t));
     }
 
     private void collect(Invocation invocation, 
Consumer<MetricsCollectExecutor> execute) {

Reply via email to