Page MenuHomePhabricator

Make instance migration error clearer
ClosedPublic

Authored by dgibson on Thu, Jan 14, 2:55 AM.

Details

Summary

Exception chaining is producing this monstrosity: https://dagster.phacility.com/P145

with the important bit (that you need to migrate) in the middle of a gross SQL sandwich.

Disabling exception chaining gives us this instead, which is IMO much clearer: https://dagster.phacility.com/P146

Test Plan

migrate from 0.9.0 to 0.10.0, bask in the clarity of the new error message

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.