Skip to content

Fix split streaming#4042

Merged
roman-khimov merged 2 commits into
masterfrom
fix-split-streaming
Jun 24, 2026
Merged

Fix split streaming#4042
roman-khimov merged 2 commits into
masterfrom
fix-split-streaming

Conversation

@End-rey

@End-rey End-rey commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Closes #4041.

End-rey added 2 commits June 24, 2026 16:11
Handle virtual children in pipelined split assembly by falling back to the
regular recursive get path. Preserve headers in payload-only mode so child
validation still works.

Closes #4041.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
Mark the response as started when pipelined child payload is written, matching
the non-pipelined streaming paths.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 26.75%. Comparing base (6dd5ff9) to head (bc37f22).

Files with missing lines Patch % Lines
pkg/services/object/get/exec.go 57.14% 3 Missing ⚠️
pkg/services/object/get/assemble_pipeline.go 90.90% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4042      +/-   ##
==========================================
+ Coverage   26.72%   26.75%   +0.02%     
==========================================
  Files         688      688              
  Lines       46634    46652      +18     
==========================================
+ Hits        12463    12480      +17     
- Misses      33038    33039       +1     
  Partials     1133     1133              

☔ 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.


if s.se.status != statusOK {
if s.se.status == statusVIRTUAL {
if exec.copyChild(s.id, s.rng, checkHdr) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Likely can be optimized, but works as a fix.

@roman-khimov roman-khimov merged commit 2e51591 into master Jun 24, 2026
20 of 22 checks passed
@roman-khimov roman-khimov deleted the fix-split-streaming branch June 24, 2026 13:27
@End-rey End-rey mentioned this pull request Jun 24, 2026
roman-khimov added a commit that referenced this pull request Jun 24, 2026
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.

S3 multipart test failure

2 participants