Skip to content
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

Adding more robust reporting for PerformHarvestJobs #34

Merged
merged 10 commits into from
Sep 14, 2022

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Sep 12, 2022

Prior to this commit we had two emails that were sent:

  1. harvest_failed: Regardless of Harvester#harvest_oai_items results, if there were sidecar items missing, we'd send an harvest_failed email. This email did not include any additional information on what sidecars were missing.
  2. harvest_succeeded: If all harvest documents had existing sidecars, we'd send a harvest_succeeded email.

Note: prior to and including this change, there's not handling of errors encountered elsewhere (e.g. invalid data, network issues, database failures, etc).

With this commit, we consolidate the harvest_failed and harvest_succeeded into harvest_set_completed. Sending the harvest_set_completed email is sent when we are done running the job; there may or may not be issues with document's related sidecars. If there are issues, we include details of those issues in the email.

Closes #185

Testing Instructions

Pending

jeremyf and others added 8 commits September 12, 2022 11:24
Prior to this commit we had two emails that were sent:

1. **harvest_failed**: Regardless of `Harvester#harvest_oai_items`
   results, if there were sidecar items missing, we'd send an
   `harvest_failed` email.  This email did not include any additional
   information on what sidecars were missing.
2. **harvest_succeeded**: If all harvest documents had existing
   sidecars, we'd send a `harvest_succeeded` email.

*Note:* prior to and including this change, there's not handling of errors
encountered elsewhere (e.g. invalid data, network issues, database
failures, etc).

With this commit, we consolidate the `harvest_failed` and
`harvest_succeeded` into `harvest_set_completed`.  That
`harvest_set_completed` means that the set completed and there may or
may not be issues with document's related sidecars.  If there are
issues, we include details of those issues in the email.

Closes #185
….com:harvard-lts/spotlight-oaipmh-resources into jeremyf---harvard-lts---CURIOSity#185-part-2
@jeremyf jeremyf marked this pull request as ready for review September 12, 2022 20:21
Copy link
Contributor

@phil-plencner-hl phil-plencner-hl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this out and it works as described in https://github.com/harvard-lts/CURIOSity/issues/185

Approved.

@phil-plencner-hl phil-plencner-hl merged commit 19fd7dc into main Sep 14, 2022
@phil-plencner-hl phil-plencner-hl deleted the jeremyf---harvard-lts---CURIOSity#185-part-2 branch September 14, 2022 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants