You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We serialize datetimes in three places now, in dbtClassMixin, EventSerialization, and in dbt.events.functions, where we serialize the logging time.
Create a utility function and use it consistently. Also probably we should separate out the DatetimeSerialization code in dbtClassMixin and EventSerialization and just have one class.
The text was updated successfully, but these errors were encountered:
gshank
added
the
tech_debt
Behind-the-scenes changes, with little direct impact on end-user functionality
label
Jan 21, 2022
github-actionsbot
changed the title
Standardize serialization of datetimes
[CT-82] Standardize serialization of datetimes
Jan 21, 2022
This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please remove the stale label or comment on the issue, or it will be closed in 7 days.
Although we are closing this issue as stale, it's not gone forever. Issues can be reopened if there is renewed community interest; add a comment to notify the maintainers.
We serialize datetimes in three places now, in dbtClassMixin, EventSerialization, and in dbt.events.functions, where we serialize the logging time.
Create a utility function and use it consistently. Also probably we should separate out the DatetimeSerialization code in dbtClassMixin and EventSerialization and just have one class.
The text was updated successfully, but these errors were encountered: