Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

[Android] WorkManagerに指定しているExistingPeriodicWorkPolicyの変更 #21

Closed
keiji opened this issue Feb 17, 2021 · 3 comments
Closed
Labels
released リリースが完了したもの

Comments

@keiji
Copy link
Collaborator

keiji commented Feb 17, 2021

Android版のバックグラウンド処理(WorkManager)においてExistingPeriodicWorkPolicy ExistingPeriodicWorkPolicy.Replaceの値を指定しています。

workManager.EnqueueUniquePeriodicWork("exposurenotification",
ExistingPeriodicWorkPolicy.Replace,
workRequest);

これはGoogleが提供しているドキュメントExposure Notifications implementation guideに基づいた記述と推測しますが、当該ドキュメントは2020年9月に更新されており、現在はExistingPeriodicWorkPolicyにはExistingPeriodicWorkPolicy.KEEPを指定するように記述が変更されています。

// Use a unique work to avoid multiple workers.
workManager.enqueueUniquePeriodicWork(
    PROVIDE_KEY_WORK_NAME,
    ExistingPeriodicWorkPolicy.KEEP,
    workRequest
)

ドキュメントの更新履歴によれば、ExistingPeriodicWorkPolicy.Replaceを指定した場合、WorkManagerがシステムのJobSchedulerにジョブを追加しない問題が起こることがある。とのことです。

This resolves an issue with WorkManager sometimes not adding a job to the system job scheduler.

最新のドキュメントの内容に基づいて、当該部分をExistingPeriodicWorkPolicy.KEEPに変更することを強くお勧めします。

@keiji
Copy link
Collaborator Author

keiji commented Feb 17, 2021

Issueとは直接関係ないのでコメントとして記載します。

現在のCOCOAの状況を大変、憂慮しています。

このIssueを見ている関係者の方は居られないのかもしれません。しかし、もし居られるならば、今一度、オープンソースのやり方でCOCOAを継続的に改善する方向へ舵を切ることを何卒ご検討ください。

もしそうなったならば、ぼくはAndroidアプリ開発に携わるソフトウェア開発者の一人として、お力になれるものと確信しています。

@heykuro
Copy link
Collaborator

heykuro commented Mar 4, 2021

@keiji さん、遅くなりましたがご指摘ありがとうございます。

#25 #31 の対応に併せて、こちらも対応予定です。

ExistingPeriodicWorkPolicy.KEEP

に修正します。

また、コメントもありがとうございます。理想的なオープンソース開発にはまだ至れていないと思いますが、より良い開発プロセスを目指して進めていきたいと考えておりますので、今後ともよろしくお願いいたします。

@keiji keiji added the in progress 現在対応中、または対応準備を開始しているもの label Mar 19, 2021
@keiji keiji added ready-for-release マージが済み、リリース準備が完了しているもの and removed in progress 現在対応中、または対応準備を開始しているもの labels Mar 31, 2021
@keiji keiji added this to 接触確認関連 in v1.2.3 - 202104 Apr 8, 2021
@keiji
Copy link
Collaborator Author

keiji commented Apr 21, 2021

v.1.2.3で ExistingPeriodicWorkPolicy.KEEP に変更されましたので本Issueはクローズします。

@keiji keiji closed this as completed Apr 21, 2021
@keiji keiji added released リリースが完了したもの and removed ready-for-release マージが済み、リリース準備が完了しているもの labels Apr 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
released リリースが完了したもの
Projects
No open projects
v1.2.3 - 202104
接触通知関連 / Exposure Notification
Development

No branches or pull requests

2 participants