Page MenuHomePhabricator

[report service] Move ReportContent to lib
ClosedPublic

Authored by michal on Dec 19 2023, 6:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 1, 6:15 PM
Unknown Object (File)
Fri, Feb 28, 11:13 PM
Unknown Object (File)
Fri, Feb 28, 11:13 PM
Unknown Object (File)
Mon, Feb 24, 7:01 AM
Unknown Object (File)
Thu, Feb 20, 7:24 PM
Unknown Object (File)
Feb 6 2025, 9:46 AM
Unknown Object (File)
Jan 25 2025, 9:00 AM
Unknown Object (File)
Jan 25 2025, 8:52 AM
Subscribers

Details

Summary

ENG-5869

We would like to store backup logs either in dynamodb, or if their size is too big, store them in blob service. Report service already implements a similar functionality so this diff moves the struct to comm-lib. Some small changes were made:

  • Added a new() function so there's less chance that people will create the in-blob variant manually
  • Renaming and added some more comments
  • Now during (de-)serialization to dynamodb values the caller must pass the attribute names
Test Plan

cargo check in report service

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable