Page MenuHomePhabricator

[core] rework engine requirments checks
ClosedPublic

Authored by alangenfeld on Tue, Jan 14, 9:53 PM.

Details

Summary
  • add a few things to the init executor context
  • drop the separate requirements_check
  • create a single check_cross_process_constraints check and migrate to it

Resolves #1649

Test Plan

added test case

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.Tue, Jan 14, 9:53 PM
alangenfeld edited the summary of this revision. (Show Details)Tue, Jan 14, 9:55 PM
max accepted this revision.Tue, Jan 14, 10:32 PM
This revision is now accepted and ready to land.Tue, Jan 14, 10:32 PM
This revision was automatically updated to reflect the committed changes.