Paths

Table of Contentst

-
services/backup/docker-server/contents/server/src/Reactors/server/
-
backup/
-
docker-server/
-
contents/
-
server/
-
src/
-
Reactors/
-
server/
1/1
SendLogReactor.h

Differential D3647

[services] Backup - Send log - add backup id and log hash states
ClosedPublic
Actions

Authored by • karol on Apr 7 2022, 7:21 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Mar 15, 5:53 AM

	Unknown Object (File)
	Mon, Mar 3, 6:59 PM

	Unknown Object (File)
	Sun, Feb 23, 2:41 PM

	Unknown Object (File)
	Feb 17 2025, 4:26 AM

	Unknown Object (File)
	Feb 15 2025, 7:17 AM

	Unknown Object (File)
	Feb 14 2025, 11:00 PM

	Unknown Object (File)
	Feb 13 2025, 11:50 AM

	Unknown Object (File)
	Feb 13 2025, 4:44 AM

Subscribers

Details

Reviewers

tomek
• max

Commits

rCOMM430640213fac: [services] Backup - Send log - add backup id and log hash states

Summary

Depends on D3646

Adding two missing states for the reactor: BACKUP_ID and LOG_HASH. along with the cases in the reading request method's body.

Test Plan

cd services
yarn run-backup-service

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• karol created this revision.Apr 7 2022, 7:21 AM

Herald added subscribers: • abosh, • benschac, atul and 3 others. · View Herald TranscriptApr 7 2022, 7:21 AM

• karol edited the summary of this revision. (Show Details)Apr 7 2022, 7:27 AM

• karol edited the test plan for this revision. (Show Details)

• karol added reviewers: tomek, • max.

• karol added a parent revision: D3646: [services] Backup - Send log - initialize put reactor method.

• karol added a child revision: D3648: [services] Backup - Send log - add reactor state mutex.Apr 7 2022, 7:29 AM

Harbormaster completed remote builds in B7967: Diff 11171.Apr 7 2022, 8:32 AM

• karol requested review of this revision.Apr 7 2022, 8:32 AM

tomek accepted this revision.Apr 8 2022, 1:23 AM

tomek added inline comments.

services/backup/docker-server/contents/server/src/Reactors/server/SendLogReactor.h
94–104 ↗	(On Diff #11171)	It is ok to introduce new, unimplemented states, but it would be better if we added them one-by-one, with the implementation. Also, `this->state = State::LOG_CHUNK;` is deleted... shouldn't we replace it with `this->state = State::BACKUP_ID;`?

This revision is now accepted and ready to land.Apr 8 2022, 1:23 AM

• karol added inline comments.Apr 13 2022, 4:19 AM

services/backup/docker-server/contents/server/src/Reactors/server/SendLogReactor.h
94–104 ↗	(On Diff #11171)	It is ok to introduce new, unimplemented states, but it would be better if we added them one-by-one, with the implementation. I understand. I'm going to try to remember about this next time. Also, `this->state = State::LOG_CHUNK;` is deleted... shouldn't we replace it with `this->state = State::BACKUP_ID;`? t is added in D3651. Here we return `unimplemented` anyway so there's no point in caring about the correctness of the logic as it's not gonna work here yet anyway.

Closed by commit rCOMM430640213fac: [services] Backup - Send log - add backup id and log hash states (authored by • karol). · Explain WhyApr 13 2022, 4:20 AM

This revision was automatically updated to reflect the committed changes.

• karol added a commit: rCOMM430640213fac: [services] Backup - Send log - add backup id and log hash states.

Revision Contents
Changeset List

Path

Size

services/

backup/

docker-server/

contents/

server/

src/

Reactors/

server/

SendLogReactor.h

13 lines

Diff 11394

services/backup/docker-server/contents/server/src/Reactors/server/SendLogReactor.h

	::-webkit-scrollbar {			::-webkit-scrollbar {
	width: 15px;			width: 15px;
	}			}
	::-webkit-scrollbar-track-piece {			::-webkit-scrollbar-track-piece {
	background-color: var(--shades-black-80);			background-color: var(--shades-black-75);
				ginsuAuthorUnsubmitted Done Inline Actions This will be fixed with https://linear.app/comm/issue/ENG-5687/desktop-app-scrollbar ginsu: This will be fixed with https://linear.app/comm/issue/ENG-5687/desktop-app-scrollbar
	}			}
	::-webkit-scrollbar-thumb {			::-webkit-scrollbar-thumb {
	background-color: var(--shades-black-60);			background-color: var(--shades-black-50);
	border: 3px var(--shades-black-80) solid;			border: 3px var(--shades-black-75) solid;
	border-radius: 10px;			border-radius: 10px;
	}			}