Page MenuHomePhabricator

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

Authored by bartek on Nov 30 2022, 7:02 AM.
Tags
None
Referenced Files
F2829682: D5768.id19013.diff
Fri, Sep 27, 5:09 PM
Unknown Object (File)
Sat, Sep 21, 5:14 PM
Unknown Object (File)
Thu, Sep 19, 11:28 PM
Unknown Object (File)
Thu, Sep 19, 11:28 PM
Unknown Object (File)
Thu, Sep 19, 11:27 PM
Unknown Object (File)
Thu, Sep 19, 11:24 PM
Unknown Object (File)
Wed, Sep 4, 10:27 PM
Unknown Object (File)
Aug 22 2024, 7:18 PM
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