Page MenuHomeElementl

crag migration guide
ClosedPublic

Authored by sandyryza on Mon, Jul 12, 5:00 PM.

Details

Summary

Depends on D8823.

Test Plan

bk, manual inspection

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.Mon, Jul 12, 5:24 PM
Harbormaster failed remote builds in B33559: Diff 41422!
Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Jul 12, 7:50 PM
Harbormaster failed remote builds in B33568: Diff 41437!

test failure is unrelated

this sort of dances around the graph -> job coercion bit, which is admittedly hard to thread the needle on. Maybe when Job is introduced you could mention the graph case works by coercing in to one or something like that? Maybe something for a follow up iteration.

should this get a nav entry?

docs/content/guides/dagster/graph_job_op.mdx
4–8

rm todo - docs are linked down here

212–215

triple-tick block for this?

278–297

should we show using the yaml parsing fn to feed config= ? I think that might be a common thing when moving from preset

301–310

yikes - @config_mapping where you at

i think this is a fine place to start

This revision is now accepted and ready to land.Mon, Jul 12, 8:46 PM
sandyryza added inline comments.
docs/content/guides/dagster/graph_job_op.mdx
212–215

I had that originally, but I believe yarn snapshot removed it.

301–310

yeah :(

This revision was automatically updated to reflect the committed changes.