[backup] Extract function to query DDB index
ClosedPublic
Actions

Authored by bartek on Aug 5 2024, 4:46 AM.

Details

Reviewers

kamil
marcin
varun

Commits

rCOMM32dbc25d2be6: [backup] Extract function to query DDB index

Summary

Noticed that the same query is used in two places, and I'm going to introduce third one.

Test Plan

Created two backups. Made sure that behavior hasn't been changed: the latest is retrieved, and the old is removed.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Aug 5 2024, 4:46 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 5 2024, 4:46 AM

bartek added a child revision: D12971: [backup] Add endpoint to delete user data.Aug 5 2024, 4:49 AM

bartek mentioned this in D12973: [identity] Delete Backup user data on account deletion.Aug 5 2024, 4:54 AM

bartek published this revision for review.Aug 5 2024, 5:08 AM

kamil accepted this revision.Aug 5 2024, 5:41 AM

This revision is now accepted and ready to land.Aug 5 2024, 5:41 AM

This revision was landed with ongoing or failed builds.Aug 5 2024, 6:27 AM

Closed by commit rCOMM32dbc25d2be6: [backup] Extract function to query DDB index (authored by bartek). · Explain Why

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM32dbc25d2be6: [backup] Extract function to query DDB index.

bartek mentioned this in rCOMM7d827184c069: [identity] Delete Backup user data on account deletion.

Harbormaster completed remote builds in B30907: Diff 43087.Aug 5 2024, 6:41 AM

Revision Contents
Changeset List

Path

Size

services/

backup/

src/

database/

mod.rs

110 lines

Diff 43087

View Options

services/backup/src/database/mod.rs

[backup] Extract function to query DDB indexClosedPublicActions