diff --git a/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/thread/JobLosedMonitorHelper.java b/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/thread/JobLosedMonitorHelper.java index 488443f9..b5575253 100644 --- a/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/thread/JobLosedMonitorHelper.java +++ b/jero-boot/jero-cloud-module/jero-cloud-xxljob/src/main/java/com/xxl/job/admin/core/thread/JobLosedMonitorHelper.java @@ -19,7 +19,7 @@ import java.util.concurrent.TimeUnit; */ public class JobLosedMonitorHelper { private static Logger logger = LoggerFactory.getLogger(JobLosedMonitorHelper.class); - + private static JobLosedMonitorHelper instance = new JobLosedMonitorHelper(); public static JobLosedMonitorHelper getInstance(){ return instance; @@ -30,11 +30,7 @@ public class JobLosedMonitorHelper { private Thread monitorThread; private volatile boolean toStop = false; public void start(){ - monitorThread = new Thread(new Runnable() { - - @Override - public void run() { - + monitorThread = new Thread(()->{ // monitor while (!toStop) { try { @@ -42,7 +38,7 @@ public class JobLosedMonitorHelper { Date losedTime = DateUtil.addMinutes(new Date(), -10); List losedJobIds = XxlJobAdminConfig.getAdminConfig().getXxlJobLogDao().findLostJobIds(losedTime); - if (losedJobIds!=null && losedJobIds.size()>0) { + if (losedJobIds!=null && !losedJobIds.isEmpty()) { for (Long logId: losedJobIds) { XxlJobLog jobLog = new XxlJobLog(); @@ -61,7 +57,6 @@ public class JobLosedMonitorHelper { logger.error(">>>>>>>>>>> xxl-job, job fail monitor thread error:{}", e); } } - try { TimeUnit.SECONDS.sleep(60); } catch (Exception e) { @@ -69,12 +64,8 @@ public class JobLosedMonitorHelper { logger.error(e.getMessage(), e); } } - } - logger.info(">>>>>>>>>>> xxl-job, JobLosedMonitorHelper stop"); - - } }); monitorThread.setDaemon(true); monitorThread.setName("xxl-job, admin JobLosedMonitorHelper");