Page MenuHomePhabricator

[services] Backup - Add Attachment Reactor Logic
ClosedPublic

Authored by karol on May 26 2022, 3:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 16, 6:29 PM
Unknown Object (File)
Sun, Jun 16, 6:29 PM
Unknown Object (File)
Sun, Jun 16, 6:29 PM
Unknown Object (File)
Sun, Jun 16, 6:29 PM
Unknown Object (File)
Sat, Jun 15, 3:31 PM
Unknown Object (File)
Fri, Jun 14, 4:46 AM
Unknown Object (File)
May 29 2024, 10:37 AM
Unknown Object (File)
May 29 2024, 10:37 AM

Details

Summary

Depends on D4134

Adding AddAttachments logic. This is a Unary Reactor which means it goes only for one cycle, the response is empty, so it effectively just receives the data on the server and then returns a status.

logID is optional. If it's an empty string, the attachments are appended to the backup item.

Test Plan
  • terminal 1
cd services
yarn run-blob-service-dev-mode
  • terminal 2
cd services
yarn run-backup-service-dev-mode
  • terminal 3
git clone https://github.com/karol-bisztyga/grpc-playground.git
cd grpc-playground
git checkout backup-async
./build.sh
./cmake/build/bin/client
  • in terminal 3 create a new backup with n
  • check the database and see the empty string in attachmentHolders in the newly created item
  • add new attachments: in terminal 3 use b
  • check the database again and see new attachments in attachmentHolders in the newly created item
  • do a similar thing for the logs: first, use l and then k

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

karol edited the test plan for this revision. (Show Details)
karol added a reviewer: tomek.
This revision is now accepted and ready to land.May 27 2022, 6:03 AM
karol edited the summary of this revision. (Show Details)
karol edited the test plan for this revision. (Show Details)

update