From 2f312486a1e24786569cb6d1f247a4348cd29f72 Mon Sep 17 00:00:00 2001 From: Mygod Date: Fri, 5 May 2017 13:43:19 +0800 Subject: [PATCH] Avoid too many reschedules correctly --- .../scala/com/github/shadowsocks/acl/AclSyncJob.scala | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/mobile/src/main/scala/com/github/shadowsocks/acl/AclSyncJob.scala b/mobile/src/main/scala/com/github/shadowsocks/acl/AclSyncJob.scala index 8dcc6749..c532039c 100644 --- a/mobile/src/main/scala/com/github/shadowsocks/acl/AclSyncJob.scala +++ b/mobile/src/main/scala/com/github/shadowsocks/acl/AclSyncJob.scala @@ -46,8 +46,6 @@ object AclSyncJob { } class AclSyncJob(route: String) extends Job { - var rescheduled = 0 - override def onRunJob(params: Params): Result = { try { //noinspection JavaAccessorMethodCalledAsEmptyParen @@ -57,11 +55,7 @@ class AclSyncJob(route: String) extends Job { } catch { case e: IOException => e.printStackTrace() - rescheduled += 1 - if (rescheduled < AclSyncJob.MAX_RESCHEDULE) - Result.RESCHEDULE - else - Result.FAILURE + if (params.getFailureCount < AclSyncJob.MAX_RESCHEDULE) Result.RESCHEDULE else Result.FAILURE case e: Exception => // unknown failures, probably shouldn't retry e.printStackTrace() Result.FAILURE