Skip to content

Improve EC GET parallelism - #4101

Draft
cthulhu-rider wants to merge 2 commits into
masterfrom
ec-get-parallel
Draft

Improve EC GET parallelism#4101
cthulhu-rider wants to merge 2 commits into
masterfrom
ec-get-parallel

Conversation

@cthulhu-rider

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 122 lines in your changes missing coverage. Please review.
✅ Project coverage is 27.19%. Comparing base (603b682) to head (6f866b8).

Files with missing lines Patch % Lines
pkg/services/object/get/ec.go 0.00% 66 Missing ⚠️
pkg/services/object/get.go 0.00% 56 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4101      +/-   ##
==========================================
- Coverage   27.21%   27.19%   -0.02%     
==========================================
  Files         676      676              
  Lines       46221    46257      +36     
==========================================
+ Hits        12578    12580       +2     
- Misses      32435    32469      +34     
  Partials     1208     1208              

☔ View full report in Codecov by Harness.
📢 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.

It's planned to modify concurrent code for multiple parts. This will
allow to not touch the code of a simple corner case.

Signed-off-by: Leonard Liubich <leonard@morphbits.io>
Previously, secondary EC data chunks were blocked until the first one
was completely received and sent to the response stream. However, in the
current implementation, only the first part's and parent payload lengths
are required to request secondary chunks (using ranged GET). Both of
these values are obtained from header of the first part received using
GET. Therefore, secondary routines may not wait for first part's payload
to be fully copied and be unlocked earlier. This acceleration will be
more noticeable with the growth of the object size (and consequently its
data parts).

Signed-off-by: Leonard Liubich <leonard@morphbits.io>
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