Page MenuHomePhabricator

[core] rework engine requirments checks
ClosedPublic

Authored by alangenfeld on Jan 14 2020, 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
Branch
celery (branched from master)
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

alangenfeld created this revision.Jan 14 2020, 9:53 PM
alangenfeld edited the summary of this revision. (Show Details)Jan 14 2020, 9:55 PM
max accepted this revision.Jan 14 2020, 10:32 PM
This revision is now accepted and ready to land.Jan 14 2020, 10:32 PM
This revision was automatically updated to reflect the committed changes.