-
Notifications
You must be signed in to change notification settings - Fork 3
VED-991 Integrate DQ reporting into FHIR API and Batch #1086
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
VED-991 Integrate DQ reporting into FHIR API and Batch #1086
Conversation
|
This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket: VED-991 |
| logger.info(f"Restarted Process (log up to first 10): {total_rows_processed_count + 1}") | ||
|
|
||
| # Submit data quality report | ||
| data_quality_reporter.generate_and_send_report(row) |
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.
This hopefully won't be an issue, but we will send a duplicate report if we retry processing with a different file encoding
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.
This Lambda is hard to follow and should definitely be refactored if there is ever some time.
However, one of the benefits of how it has been written is if it falls over for that reason, it only continues processing from where it left off. See line 134. Therefore, proposing to keep this as is.
|
ebdb5df
into
staging/VED-963-data-quality-reports



Summary
Integrates the data quality reports into the batch processing system and the Create and Update endpoints of the Imms FHIR API.
Please refer to the ticket for test evidence of both the logs and the reports which are produced and sent to S3.
Reviews Required
Review Checklist
ℹ️ This section is to be filled in by the reviewer.