New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: when queue cannot find task, expect queue runs next task #397
Conversation
Sample app builds 📱Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.
|
Codecov Report
@@ Coverage Diff @@
## main #397 +/- ##
==========================================
+ Coverage 56.11% 59.16% +3.05%
==========================================
Files 103 119 +16
Lines 1153 4602 +3449
==========================================
+ Hits 647 2723 +2076
- Misses 506 1879 +1373
|
@@ -95,7 +95,7 @@ public class CioQueueRunRequest: QueueRunRequest { | |||
|
|||
// The task failed to execute like a HTTP failure. Update `lastFailedTask`. | |||
updateWhileLoopLogicVariables(didTaskFail: true, taskJustExecuted: nextTaskToRunInventoryItem) | |||
break | |||
continue // quit loop early and move to next task. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this possibly fix the bug that the customer reported ?
## [2.8.5](2.8.4...2.8.5) (2023-10-27) ### Bug Fixes * when queue cannot find task, expect queue runs next task ([#397](#397)) ([01ea7a6](01ea7a6))
Through reviewing a set of SDK debug logs recently, I noticed the queue can get into an edge case that prevents it from ever finishing. Blocking the queue from running any more tasks while the app is in memory.
I was able to reproduce the issue through a test and then implemented a bug fix.