Periodic work is not triggered more than once on 1.0.0-alpha07 [112859683]

Fixed

Bug

Status Update

No update yet.

Description

ne...@gmail.com

created issue #1

Aug 20, 2018 10:08PM

Component used: WorkManager
Version used: 1.0.0-alpha07
Devices/Android versions reproduced on: Pixel 2, Android 9 Pie

The work request below is executed once but not periodically on version alpha07, this works correctly and periodically on version alpha06

val constraints = androidx.work.Constraints.Builder()
.setRequiredNetworkType(NetworkType.CONNECTED)
.build() // Run only when network is available
val workRequest =
PeriodicWorkRequestBuilder<ReportUploader>(REPORTING_REPEAT_INTERVAL_IN_MINUTES, TimeUnit.MINUTES)
.addTag(REPORTER_TAG)
.setConstraints(constraints)
.setBackoffCriteria(BackoffPolicy.EXPONENTIAL, 10, TimeUnit.SECONDS)
.build() // Run every 15 mins
workManager
.enqueueUniquePeriodicWork(REPORTER_TAG, ExistingPeriodicWorkPolicy.KEEP, workRequest)

Comments

ne...@gmail.com <ne...@gmail.com> #2Aug 20, 2018 10:10PM

I forgot to mention that my Woker returns Result.SUCCESS always.

ra...@google.com <ra...@google.com> Aug 21, 2018 11:33PM

Assigned to ra...@google.com.

su...@google.com <su...@google.com> Aug 21, 2018 11:45PM

Accepted by su...@google.com.

ap...@google.com <ap...@google.com> #3Aug 22, 2018 12:44AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 05f2d67cc04c5f76bee122b9e1e23ca1bf61a326
Author: Sumir Kataria <sumir@google.com>
Date: Tue Aug 21 16:36:05 2018

Fix periodic work so it executes repeatedly.

Processor.onExecuted is the wrong place for cancelling things in
other schedulers. It doesn't have the proper information about
what needs to be cancelled. For example, PeriodicWork should NOT
be cancelled. This should be done right before Schedulers.schedule
at the end of WorkerWrapper.

Bug: 112859683
Test: Added new tests for API 22-, manually tested using
integration app for 23+ since JobScheduler enforces a 15 minute
limit between executions

Change-Id: Ia0e6e17367fbc2493339b9b1275453267c571580

M work/workmanager/src/androidTest/java/androidx/work/impl/WorkManagerImplTest.java
M work/workmanager/src/main/java/androidx/work/impl/Processor.java
M work/workmanager/src/main/java/androidx/work/impl/WorkerWrapper.java

https://android-review.googlesource.com/735827

https://goto.google.com/android-sha1/05f2d67cc04c5f76bee122b9e1e23ca1bf61a326

lo...@gmail.com <lo...@gmail.com> #4Aug 22, 2018 03:24PM

I am experiencing the same problem using the following code:

val myWorker = PeriodicWorkRequestBuilder<MyWorker>(15, TimeUnit.MINUTES)

myWorker.addTag(NEWSFEED_SYNC_WORKER_ID)

val constraints = Constraints.Builder()
.setRequiredNetworkType(NetworkType.CONNECTED)
.build()
myWorker.setConstraints(constraints)

WorkManager.getInstance().enqueue(myWorker.build())

su...@google.com <su...@google.com> #5Aug 22, 2018 04:03PM

Marked as fixed.

The fix will be available in alpha08.

ta...@gmail.com <ta...@gmail.com> #6Oct 6, 2018 09:23AM

I'm facing this in alpha09 on Android Pie

ta...@gmail.com <ta...@gmail.com> #7Oct 7, 2018 09:48AM

Update: alpha09 works fine in pre-Oreo with a minimum interval of 15 minutes. Post Oreo, never gets triggered. Haven't yet checked the interference of battery optimization. Will update on the same soon.

ah...@gmail.com <ah...@gmail.com> #8Nov 4, 2018 09:36AM

Still facing this in alpha10 on Oreo and Pie

ar...@gmail.com <ar...@gmail.com> #9Nov 22, 2018 06:10PM

solved in alpha 11 i have not this problem but some times WorkManger run 2 times instead one Worker class in Periodic work

pr...@gmail.com <pr...@gmail.com> #10Nov 24, 2018 09:32AM

Is there any other way to use periodic job scheduler except WorkManager,Job Creator?