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 85b3374fa [SCB-2462] Fix equals in TrafficMarker (#2797)
85b3374fa is described below
commit 85b3374fac1019fa5ba3bf792fd83974633da857
Author: ZhangJian He <[email protected]>
AuthorDate: Mon Apr 25 20:20:42 2022 +0800
[SCB-2462] Fix equals in TrafficMarker (#2797)
---
ci/spotbugs/exclude.xml | 4 ----
.../java/org/apache/servicecomb/governance/marker/TrafficMarker.java | 3 +--
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/ci/spotbugs/exclude.xml b/ci/spotbugs/exclude.xml
index e0b6e6c48..d37d28e2b 100644
--- a/ci/spotbugs/exclude.xml
+++ b/ci/spotbugs/exclude.xml
@@ -155,10 +155,6 @@
<Bug pattern="RV_NEGATING_RESULT_OF_COMPARETO"/>
</Match>
- <Match>
- <Bug pattern="SA_FIELD_SELF_COMPARISON"/>
- </Match>
-
<Match>
<Bug pattern="SE_BAD_FIELD"/>
</Match>
diff --git
a/governance/src/main/java/org/apache/servicecomb/governance/marker/TrafficMarker.java
b/governance/src/main/java/org/apache/servicecomb/governance/marker/TrafficMarker.java
index 9dbaf4d73..a578d3f9a 100644
---
a/governance/src/main/java/org/apache/servicecomb/governance/marker/TrafficMarker.java
+++
b/governance/src/main/java/org/apache/servicecomb/governance/marker/TrafficMarker.java
@@ -52,7 +52,6 @@ public class TrafficMarker extends Configurable {
}
public boolean checkMatch(GovernanceRequest governanceRequest,
RequestProcessor requestProcessor) {
- return this.matches.stream().anyMatch(match ->
- this.name.equals(name) && requestProcessor.match(governanceRequest,
match));
+ return this.matches.stream().anyMatch(match ->
requestProcessor.match(governanceRequest, match));
}
}