[reports-service] Introduce database client
ClosedPublic
Actions

Authored by bartek on Aug 25 2023, 10:10 AM.

Details

Reviewers

michal
varun
• patryk
• jon

Commits

rCOMMc31b15da92b3: [reports-service] Introduce database client

Summary

Similiar to other services, created a struct representing DynamoDB client.
Created basic interfaces to store and retrieve reports. They're very simple but sufficient for now.

Depends on D8939, D8895

Test Plan

Storing reports was tested in D8895, other methods are tested together with HTTP interface (later in the stack)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Aug 25 2023, 10:10 AM

bartek held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptAug 25 2023, 10:10 AM

Herald added subscribers: tomek, ashoat. · View Herald Transcript

Harbormaster completed remote builds in B22078: Diff 30306.Aug 25 2023, 10:28 AM

bartek published this revision for review.Aug 25 2023, 11:21 AM

• jon accepted this revision.Aug 25 2023, 11:37 PM

• jon added inline comments.

services/reports/src/database/client.rs
58 ↗	(On Diff #30306)	should this go in constants?
102 ↗	(On Diff #30306)	Potentially avoid forcing a Vec construction here if we take `IntoIter<ReportItem>`. But feel strongly about it, as DDB deals with Vec directly.