Page MenuHomeElementl

rename StoredEventRecord -> EventLogRecord
ClosedPublic

Authored by yuhan on Jun 16 2021, 11:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 23, 2:03 AM
Unknown Object (File)
Wed, Jan 4, 8:52 PM
Unknown Object (File)
Mon, Jan 2, 6:31 AM
Unknown Object (File)
Nov 26 2022, 9:55 PM
Unknown Object (File)
Nov 22 2022, 2:57 PM
Unknown Object (File)
Nov 16 2022, 5:59 PM
Unknown Object (File)
Nov 13 2022, 11:03 PM
Unknown Object (File)
Nov 13 2022, 9:19 PM
Subscribers
None

Details

Summary

depends on D8422

Test Plan

bk
pipeline failure sensor works

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.Jun 17 2021, 12:05 AM
Harbormaster failed remote builds in B32259: Diff 39730!
yuhan added a child revision: Restricted Differential Revision.Jun 17 2021, 1:17 AM

Would feel more comfortable with this being EventLogRecord (as per the discussion in D8422)

This revision now requires changes to proceed.Jun 17 2021, 2:23 PM
yuhan retitled this revision from rename StoredEventRecord -> EventRecord to rename StoredEventRecord -> EventLogRecord.Jun 17 2021, 8:50 PM
yuhan retitled this revision from rename StoredEventRecord -> EventLogRecord to rename StoredEventRecord -> EventRecord.

EventLogRecord

This revision is now accepted and ready to land.Jun 17 2021, 9:25 PM
yuhan retitled this revision from rename StoredEventRecord -> EventRecord to rename StoredEventRecord -> EventLogRecord.Jun 17 2021, 9:26 PM
This revision was landed with ongoing or failed builds.Jun 18 2021, 5:10 PM
This revision was automatically updated to reflect the committed changes.