Page MenuHomePhabricator
Differential D6312

[services][backup] Remove old C++ implementation
ClosedPublic
Actions

Authored by bartek on Jan 19 2023, 11:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 5:16 PM
Unknown Object (File)
Mon, Nov 11, 11:57 PM
Unknown Object (File)
Mon, Nov 11, 11:56 PM
Unknown Object (File)
Mon, Nov 11, 11:56 PM
Unknown Object (File)
Sat, Nov 9, 2:31 AM
Unknown Object (File)
Sat, Nov 9, 2:29 AM
Unknown Object (File)
Sat, Nov 9, 2:26 AM
Unknown Object (File)
Sat, Nov 9, 1:19 AM
View All 83 Files
Subscribers
atul
tomek

Details

Reviewers
• jon
varun
ashoat
Commits
rCOMMaf2470e1e909: [services][backup] Remove old C++ implementation
Summary

Rust implementation of Backup service is landed. The "old" C++ implementation can now be removed.

This also removes the "old blob client" that was failing in non-docker environment - see D6291

Test Plan

None. CI no longer depends on this, should not fail.

Diff Detail

Repository
rCOMM Comm
Branch
barthap/delete-backup-cpp
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Jan 19 2023, 11:46 AM
bartek held this revision as a draft.
Herald added subscribers: atul, tomek. · View Herald TranscriptJan 19 2023, 11:46 AM
Harbormaster completed remote builds in B15481: Diff 21120.Jan 19 2023, 12:01 PM
bartek published this revision for review.Jan 19 2023, 12:10 PM
bartek mentioned this in D6310: Lint all Rust projects in pre-commit.
ashoat accepted this revision.Jan 19 2023, 2:10 PM

Has blob_client been replaced or moved?

This revision is now accepted and ready to land.Jan 19 2023, 2:10 PM
bartek added a comment.Jan 20 2023, 12:28 AM
In D6312#189187, @ashoat wrote:

Has blob_client been replaced or moved?

It was replaced by a module integrated directly into the backup crate.

Closed by commit rCOMMaf2470e1e909: [services][backup] Remove old C++ implementation (authored by bartek). · Explain WhyJan 23 2023, 6:01 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMaf2470e1e909: [services][backup] Remove old C++ implementation.

Revision Contents
Changeset List

Diff 21120

View Options

.gitignore

Loading...
View Options

services/backup/old/CMakeLists.txt

Loading...
View Options

services/backup/old/blob_client/Cargo.lock

Loading...
View Options

services/backup/old/blob_client/Cargo.toml

Loading...
View Options

services/backup/old/blob_client/build.rs

Loading...
View Options

services/backup/old/blob_client/src/constants.rs

Loading...
View Options

services/backup/old/blob_client/src/get_client.rs

Loading...
View Options

services/backup/old/blob_client/src/lib.rs

Loading...
View Options

services/backup/old/blob_client/src/put_client.rs

Loading...
View Options

services/backup/old/src/BackupServiceImpl.h

Loading...
View Options

services/backup/old/src/BackupServiceImpl.cpp

Loading...
View Options

services/backup/old/src/Constants.h

Loading...
View Options

services/backup/old/src/DatabaseEntities/BackupItem.h

Loading...
View Options

services/backup/old/src/DatabaseEntities/BackupItem.cpp

Loading...
View Options

services/backup/old/src/DatabaseEntities/LogItem.h

Loading...
View Options

services/backup/old/src/DatabaseEntities/LogItem.cpp

Loading...
View Options

services/backup/old/src/DatabaseManager.h

Loading...
View Options

services/backup/old/src/DatabaseManager.cpp

Loading...
View Options

services/backup/old/src/Reactors/server/AddAttachmentsUtility.h

Loading...
View Options

services/backup/old/src/Reactors/server/AddAttachmentsUtility.cpp

Loading...
View Options

services/backup/old/src/Reactors/server/CreateNewBackupReactor.h

Loading...
View Options

services/backup/old/src/Reactors/server/CreateNewBackupReactor.cpp

Loading...
View Options

services/backup/old/src/Reactors/server/PullBackupReactor.h

Loading...
View Options

services/backup/old/src/Reactors/server/PullBackupReactor.cpp

Loading...
View Options

services/backup/old/src/Reactors/server/RecoverBackupKeyReactor.h

Loading...
View Options

services/backup/old/src/Reactors/server/SendLogReactor.h

Loading...
View Options

services/backup/old/src/Reactors/server/SendLogReactor.cpp

Loading...
View Options

services/backup/old/src/Tools.h

Loading...
View Options

services/backup/old/src/Tools.cpp

Loading...
View Options

services/backup/old/src/server.cpp

Loading...
View Options

services/backup/old/test/BackupTest.cpp

Loading...
View Options

services/backup/old/test/DatabaseManagerTest.cpp

Loading...