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)
Mon, Jul 1, 7:08 PM
Unknown Object (File)
Sat, Jun 29, 6:28 PM
Unknown Object (File)
Wed, Jun 26, 4:37 AM
Unknown Object (File)
Tue, Jun 25, 2:10 AM
Unknown Object (File)
Sat, Jun 22, 8:15 AM
Unknown Object (File)
Sat, Jun 22, 8:15 AM
Unknown Object (File)
Sat, Jun 22, 8:15 AM
Unknown Object (File)
Sat, Jun 22, 8:15 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