Page MenuHomeElementl

[dagstermill] support Failure and RetryRequested
ClosedPublic

Authored by alangenfeld on Tue, Apr 13, 4:28 PM.

Details

Summary

Papermill doesn't send over the exception object on raise so we have to use yield_event

resolves https://github.com/dagster-io/dagster/issues/4030

Test Plan

added tests

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

To be clear in vanilla solids you still have to raise these right?

To be clear in vanilla solids you still have to raise these right?

Yep - I tried to match that API here too - but PapermillExecutionException only captures the string name of the raised exception (in addition to context of the cell that raised it) so we are stuck using yield_event

This revision is now accepted and ready to land.Tue, Apr 13, 11:00 PM