From 94d55a235905f49877970da2d40a59f654e7f11f Mon Sep 17 00:00:00 2001 From: tianwenbo Date: Fri, 10 Mar 2023 16:06:37 +0800 Subject: [PATCH] =?UTF-8?q?=E3=80=90fix=20sonar=E3=80=91ExecutorRouteRound?= =?UTF-8?q?.java=E6=96=87=E4=BB=B6?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../admin/core/route/strategy/ExecutorRouteRound.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/route/strategy/ExecutorRouteRound.java b/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/route/strategy/ExecutorRouteRound.java index 936ef0ed..d43183df 100644 --- a/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/route/strategy/ExecutorRouteRound.java +++ b/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/route/strategy/ExecutorRouteRound.java @@ -14,13 +14,13 @@ import java.util.concurrent.ConcurrentMap; */ public class ExecutorRouteRound extends ExecutorRouter { - private static ConcurrentMap routeCountEachJob = new ConcurrentHashMap(); - private static long CACHE_VALID_TIME = 0; + private static ConcurrentMap routeCountEachJob = new ConcurrentHashMap<>(); + private static long cacheValidTime = 0; private static int count(int jobId) { // cache clear - if (System.currentTimeMillis() > CACHE_VALID_TIME) { + if (System.currentTimeMillis() > cacheValidTime) { routeCountEachJob.clear(); - CACHE_VALID_TIME = System.currentTimeMillis() + 1000*60*60*24; + cacheValidTime = System.currentTimeMillis() + 1000*60*60*24; } // count++ @@ -33,7 +33,7 @@ public class ExecutorRouteRound extends ExecutorRouter { @Override public ReturnT route(TriggerParam triggerParam, List addressList) { String address = addressList.get(count(triggerParam.getJobId())%addressList.size()); - return new ReturnT(address); + return new ReturnT<>(address); } }