-
Notifications
You must be signed in to change notification settings - Fork 62
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
Force job #642
Conversation
Also: add unit test, don't ignore the "force" field when retrieving the next job, and remove a duplicate line
Used to force the refresh of the cache entries, even if the job should have been skipped.
Also: fix: query first rows responses, not splits responses Also: add force argument to compute method of workers
Codecov ReportBase: 90.16% // Head: 76.20% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #642 +/- ##
===========================================
- Coverage 90.16% 76.20% -13.97%
===========================================
Files 55 14 -41
Lines 2400 353 -2047
===========================================
- Hits 2164 269 -1895
+ Misses 236 84 -152
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Note: it includes a database migration for queue (add the "force" field to all the jobs).
POST /admin/force-refresh/splits?dataset={dataset}
POST /admin/force-refresh/first-rows?dataset={dataset}&config={config}&split={split}
See #640