Page MenuHomePhabricator

Eliminate execution_epoch_time
ClosedPublic

Authored by sashank on Jan 14 2020, 10:02 PM.

Details

Summary

Deleting all references to execution_epoch_time in core, and also removing the code that added the execution_epoch_time tag to RunConfig and PipelineRun.tags

Test Plan

unit

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

sashank created this revision.Jan 14 2020, 10:02 PM
sashank edited the summary of this revision. (Show Details)Jan 14 2020, 10:31 PM
sashank added reviewers: schrockn, alangenfeld.
sashank edited the summary of this revision. (Show Details)
schrockn accepted this revision.Jan 14 2020, 10:37 PM
schrockn added inline comments.
python_modules/dagster/dagster/core/execution/api.py
330–331

❤️

This revision is now accepted and ready to land.Jan 14 2020, 10:37 PM
This revision was automatically updated to reflect the committed changes.