Page MenuHomeElementl

rename StoredEventRecord -> EventLogRecord
ClosedPublic

Authored by yuhan on Jun 16 2021, 11:20 PM.
Tags
None
Referenced Files
F2989914: D8423.id39891.diff
Thu, Jun 8, 2:38 PM
Unknown Object (File)
Sun, May 21, 12:06 PM
Unknown Object (File)
Thu, May 18, 5:58 PM
Unknown Object (File)
Thu, May 18, 12:58 PM
Unknown Object (File)
May 1 2023, 6:13 AM
Unknown Object (File)
Mar 26 2023, 8:33 AM
Unknown Object (File)
Mar 19 2023, 2:20 AM
Unknown Object (File)
Mar 16 2023, 1:14 PM
Subscribers
None

Details

Summary

depends on D8422

Test Plan

bk
pipeline failure sensor works

Diff Detail

Repository
R1 dagster
Branch
arcpatch-D8213
Lint
Lint Passed
Unit
No Test Coverage

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.