Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CT-19] Logging serialization: use mashumaro for consistency #4504

Closed
gshank opened this issue Dec 16, 2021 · 0 comments · Fixed by #4505
Closed

[CT-19] Logging serialization: use mashumaro for consistency #4504

gshank opened this issue Dec 16, 2021 · 0 comments · Fixed by #4505
Assignees
Labels
jira logging tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality
Milestone

Comments

@gshank
Copy link
Contributor

gshank commented Dec 16, 2021

The recent logging refactor used thePython dataclasses asdict method for serialization. For consistency with the rest of the codebase and for performance, switch to using mashumaro.

@gshank gshank self-assigned this Dec 16, 2021
@leahwicz leahwicz added the jira label Jan 5, 2022
@github-actions github-actions bot changed the title Logging serialization: use mashumaro for consistency [CT-19] Logging serialization: use mashumaro for consistency Jan 5, 2022
iknox-fa pushed a commit that referenced this issue Feb 8, 2022
automatic commit by git-black, original commits:
  efb890d
iknox-fa pushed a commit that referenced this issue Feb 8, 2022
automatic commit by git-black, original commits:
  3cafc9e
  efb890d
iknox-fa pushed a commit that referenced this issue Feb 8, 2022
automatic commit by git-black, original commits:
  b2aea11
  efb890d
@jtcohen6 jtcohen6 added this to the v1.1 milestone Feb 11, 2022
@jtcohen6 jtcohen6 added tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality logging labels Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira logging tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants