Page MenuHomePhabricator

[dagster-celery] handle engine errors
ClosedPublic

Authored by alangenfeld on Feb 3 2020, 10:09 PM.

Details

Summary

When we get an exception from a step - stop submitting new tasks and throw the errors all at once

Test Plan

added test

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alangenfeld created this revision.Feb 3 2020, 10:09 PM
alangenfeld updated this revision to Diff 9244.Feb 3 2020, 10:23 PM

eager exec to get test in CI

max accepted this revision.Feb 4 2020, 11:14 PM
This revision is now accepted and ready to land.Feb 4 2020, 11:14 PM
This revision was automatically updated to reflect the committed changes.