Page MenuHomePhabricator

[report service] Move ReportContent to lib
ClosedPublic

Authored by michal on Dec 19 2023, 6:13 AM.
Tags
None
Referenced Files
F2899956: D10402.id34839.diff
Sat, Oct 5, 8:40 AM
Unknown Object (File)
Sat, Sep 7, 9:55 AM
Unknown Object (File)
Sat, Sep 7, 9:25 AM
Unknown Object (File)
Sat, Sep 7, 9:25 AM
Unknown Object (File)
Fri, Sep 6, 10:31 PM
Unknown Object (File)
Sep 4 2024, 2:50 AM
Unknown Object (File)
Aug 30 2024, 12:36 PM
Unknown Object (File)
Aug 9 2024, 8:39 PM
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
No Lint Coverage
Unit
No Test Coverage