fix calculate job time cost bug

githubname1024 · githubname1024 · commit 0ea976399ce0 · 2023-02-16T10:47:22.000+08:00
diff --git a/cachecloud-web/src/main/java/com/sohu/cache/schedule/jobs/CacheBaseJob.java b/cachecloud-web/src/main/java/com/sohu/cache/schedule/jobs/CacheBaseJob.java
@@ -31,14 +31,13 @@ public abstract class CacheBaseJob implements Job, Serializable {
     public void execute(JobExecutionContext context) throws JobExecutionException {
         long start = System.currentTimeMillis();
         this.action(context);
-        long end = System.currentTimeMillis();
-        long cost = (start - end);
+        long cost = System.currentTimeMillis() - start;
         if (cost > 2000) {
             logger.warn("slowJob: job: {}, trigger: {}, cost: {} ms", context.getJobDetail().getKey(),
-                    context.getTrigger().getKey(), (end - start));
+                    context.getTrigger().getKey(), cost);
         } else {
             logger.debug("job: {}, trigger: {}, cost: {} ms", context.getJobDetail().getKey(),
-                    context.getTrigger().getKey(), (end - start));
+                    context.getTrigger().getKey(), cost);
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -31,14 +31,13 @@ public abstract class CacheBaseJob implements Job, Serializable {`
`31`	`31`	`public void execute(JobExecutionContext context) throws JobExecutionException {`
`32`	`32`	`long start = System.currentTimeMillis();`
`33`	`33`	`this.action(context);`
`34`		`- long end = System.currentTimeMillis();`
`35`		`- long cost = (start - end);`
	`34`	`+ long cost = System.currentTimeMillis() - start;`
`36`	`35`	`if (cost > 2000) {`
`37`	`36`	`logger.warn("slowJob: job: {}, trigger: {}, cost: {} ms", context.getJobDetail().getKey(),`
`38`		`- context.getTrigger().getKey(), (end - start));`
	`37`	`+ context.getTrigger().getKey(), cost);`
`39`	`38`	`} else {`
`40`	`39`	`logger.debug("job: {}, trigger: {}, cost: {} ms", context.getJobDetail().getKey(),`
`41`		`- context.getTrigger().getKey(), (end - start));`
	`40`	`+ context.getTrigger().getKey(), cost);`
`42`	`41`	`}`
`43`	`42`	`}`
`44`	`43`	`}`