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

[Ingest] Callback not working for failed jobs #2103

Open
mreyescdl opened this issue Nov 7, 2024 · 1 comment
Open

[Ingest] Callback not working for failed jobs #2103

mreyescdl opened this issue Nov 7, 2024 · 1 comment
Assignees
Labels

Comments

@mreyescdl
Copy link
Contributor

An oversight on how job notifications are processed.
Email notifications work at the Batch level, however Callback notifications are at the Job level.
This is done so that as large batches are processed, the status of these jobs are quickly processed.

However, this requires a failed Job to be acquired by the Job notification daemon.
With a Failed state a daemon can not process these jobs.

2 possible solutions

  • Process callbacks at the batch level. All jobs must complete before we start issuing callbacks
  • Architect the processing of Failed jobs by the Notification daemon somehow.
@mreyescdl mreyescdl self-assigned this Nov 7, 2024
@elopatin-uc3
Copy link
Contributor

After some initial discussion, we're thinking that the appropriate way to address this is to add a new state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants