Page MenuHomePhabricator

[sensors-7.5] rename SensorRunParams/SensorSkipData => RunRequest/SkipReason
ClosedPublic

Authored by prha on Nov 20 2020, 10:17 PM.

Diff Detail

Repository
R1 dagster
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

prha requested review of this revision.Nov 20 2020, 10:43 PM
prha retitled this revision from RFC: rename SensorRunParams/SensorSkipData => RunParams/RunSkippedData to [sensors-7.5] rename SensorRunParams/SensorSkipData => RunRequest/SkipReason.Nov 24 2020, 4:12 PM

seems reasonable to me, assuming we have something close to consensus that this is a good way forward

python_modules/dagster/dagster/core/definitions/sensor.py
47

s/body//g

48

what is an ExecutionDefinition? Is that what Job is becoming?

This revision is now accepted and ready to land.Nov 24 2020, 4:57 PM

rename execution_key => run_key