-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Attempt backport oldest first #4776
base: ja_2030417_backport_subscription_opt_to_6_4
Are you sure you want to change the base?
Attempt backport oldest first #4776
Conversation
- use a separate enum for the states - chunks have different transitions than instances. - use transactional update events for work-chunk state transitions - introduce spec-test to define behaviour of batch2 storage - replace synchronized facade with simpler ProxyUtil handler. - change job cancellation to db update query
Change work-chunk creation to be first event in state machine.
* Use java event names for work chunk transitions. * Cherry-pick d5ebd1f from rel_6_4 Avoid fetching work-chunk data (#4622) * add end time to reduction step (#4640) * add end time to reduction step * add changelog --------- Co-authored-by: Long Ma <[email protected]> (cherry picked from commit 37f5e59) * Cancel processing Provide error message in cancelled jobs, and avoid transitions in final states. * Apply tx boundary to starting job and first chunk. * cleanup * Apply tx boundary to work chunk processing * Delete BatchWorkChunk * Introduce events for job create, and chunk dequeue * Apply tx boundary to chunk handler * Move instance cancellation to database * tx boundary around stats collection and completion * tx boundary around stats collection and completion * Extend tx boundary to error, fail, and cancel * Move failure into status calc * ERROR is not an "ended" state. * Revert generics cleanup to avoid noise * Avoid sending gated chunks twice. * Make no-data path safer. Cleanup * Fix mock test for step advance. * Delete unsafe updateInstace() call * Cleanup * Changelog and notes * Fix cancel boundary. Cleanups * Cleanup * Sort mongo chunks for stable paging. Other cleanup * Document error handling * Cleanup * Update hapi-fhir-jpaserver-test-utilities/src/main/java/ca/uhn/fhir/jpa/test/Batch2JobHelper.java Co-authored-by: StevenXLi <[email protected]> --------- Co-authored-by: longma1 <[email protected]> Co-authored-by: StevenXLi <[email protected]>
* Fix intermittent in batch2 tests * Add changelog * Test fix * Resolve intermittent * Reducer changes * Test fix * Test fixes * Resolve fixme * Fix changelog
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## ja_2030417_backport_subscription_opt_to_6_4 #4776 +/- ##
==============================================================================
Coverage ? 81.30%
Complexity ? 23939
==============================================================================
Files ? 1439
Lines ? 87881
Branches ? 11833
==============================================================================
Hits ? 71453
Misses ? 11165
Partials ? 5263 ☔ View full report in Codecov by Sentry. |
* Full implementation, test, changelogs * Add changelogs * Add default method * Update hapi-fhir-storage-batch2-jobs/src/main/java/ca/uhn/fhir/batch2/jobs/export/WriteBinaryStep.java Co-authored-by: James Agnew <[email protected]> * Code Review Comments * Compilation failures --------- Co-authored-by: James Agnew <[email protected]>
No description provided.