Page MenuHomeElementl

dont re-raise errors caught by pipeline failure
ClosedPublic

Authored by alangenfeld on Tue, Apr 13, 8:26 PM.

Details

Summary

Now that PipelineFailure has the exception captured - I don't think there is a reason to re-raise it by default.

This also helps a bit with work to move resource init out of the run orchestrator

Test Plan

bk

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Apr 13, 8:45 PM
Harbormaster failed remote builds in B28785: Diff 35332!
Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Apr 13, 9:21 PM
Harbormaster failed remote builds in B28789: Diff 35338!

just lint issues which i will fix

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