Skip to content

Added CapioRemoteFile#146

Merged
marcoSanti merged 3 commits intocapio-v2from
CapioRemoteFile
Jul 23, 2025
Merged

Added CapioRemoteFile#146
marcoSanti merged 3 commits intocapio-v2from
CapioRemoteFile

Conversation

@marcoSanti
Copy link
Copy Markdown
Member

  • Added CapioRemoteFile class
  • Refactor and general cleanup

- Added CapioRemoteFile class
- Refactor and general cleanup
@marcoSanti marcoSanti marked this pull request as draft July 23, 2025 07:58
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 23, 2025

Codecov Report

Attention: Patch coverage is 19.04762% with 204 lines in your changes missing coverage. Please review.

Project coverage is 46.06%. Comparing base (4a4942d) to head (909a2aa).
Report is 1 commits behind head on capio-v2.

Files with missing lines Patch % Lines
...ommunication-service/CapioCommunicationService.hpp 0.00% 46 Missing and 43 partials ⚠️
src/server/utils/parser.hpp 34.61% 22 Missing and 46 partials ⚠️
src/server/communication-service/MTCL_backend.hpp 0.00% 17 Missing and 16 partials ⚠️
...rver/storage-service/CapioFile/CapioRemoteFile.hpp 11.11% 8 Missing ⚠️
.../server/communication-service/BackendInterface.hpp 0.00% 3 Missing ⚠️
src/server/file-manager/file_manager_impl.hpp 0.00% 0 Missing and 1 partial ⚠️
...c/server/storage-service/capio_storage_service.hpp 88.88% 1 Missing ⚠️
src/server/utils/signals.hpp 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           capio-v2     #146      +/-   ##
============================================
- Coverage     46.59%   46.06%   -0.54%     
============================================
  Files            73       76       +3     
  Lines          4088     4146      +58     
  Branches       2386     2412      +26     
============================================
+ Hits           1905     1910       +5     
- Misses         1049     1089      +40     
- Partials       1134     1147      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@marcoSanti marcoSanti force-pushed the CapioRemoteFile branch 5 times, most recently from ed05739 to cd64c30 Compare July 23, 2025 10:26
Added new class to abstract over CAPIO data plane and CAPIO control
plane. Refactored code independent of the data plane backend
@marcoSanti marcoSanti marked this pull request as ready for review July 23, 2025 10:34
@marcoSanti marcoSanti merged commit 865b141 into capio-v2 Jul 23, 2025
28 checks passed
@marcoSanti marcoSanti deleted the CapioRemoteFile branch July 23, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant