Page MenuHomePhabricator

[services][blob] Remove old C++ implementation
ClosedPublic

Authored by bartek on Nov 30 2022, 7:02 AM.
Tags
None
Referenced Files
F3259538: D5768.id19013.diff
Fri, Nov 15, 10:10 PM
F3255398: D5768.diff
Fri, Nov 15, 8:20 PM
Unknown Object (File)
Wed, Nov 6, 7:06 PM
Unknown Object (File)
Wed, Nov 6, 7:06 PM
Unknown Object (File)
Mon, Nov 4, 8:22 AM
Unknown Object (File)
Mon, Nov 4, 8:21 AM
Unknown Object (File)
Tue, Oct 22, 12:06 PM
Unknown Object (File)
Tue, Oct 22, 11:54 AM
Subscribers

Details

Summary

Removed the services/blob/old directory containing C++ impl

Resolves ENG-2306

Test Plan
  • CI should not depend on this anymore.
  • Running and building the service locally runs Rust implementation

Diff Detail

Repository
rCOMM Comm
Branch
barthap/blob-remove-cpp
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Dec 1 2022, 6:00 AM

The CI is passing for the new implementation on master and integration/performance tests are passing so I think we can go on with this one.

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