Skip to content

Conversation

DmitriyMusatkin
Copy link
Contributor

Issue #, if available:

Description of changes:
There were a couple corner cases where scheduler would request more mem from mem pool than needed, ex. for a small ranged get, the scheduler requested part sized buffer anyways. With default mem pool, the mem pool will still do part size for that case (we can optimize that later), but for custom mem pools we want to provide an option to do more accurate allocation.

Note: that for put streaming we cant always request an accurate buffer cause we really on finding eof in the stream, so its kinda of a chicken and egg problem

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov-commenter
Copy link

codecov-commenter commented Sep 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.73%. Comparing base (6c203e4) to head (c0b262a).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #563      +/-   ##
==========================================
+ Coverage   88.72%   88.73%   +0.01%     
==========================================
  Files          22       22              
  Lines        6767     6776       +9     
==========================================
+ Hits         6004     6013       +9     
  Misses        763      763              
Files with missing lines Coverage Δ
source/s3_auto_ranged_get.c 98.01% <100.00%> (+0.01%) ⬆️
source/s3_auto_ranged_put.c 92.13% <100.00%> (+0.03%) ⬆️
source/s3_client.c 91.83% <100.00%> (+<0.01%) ⬆️
source/s3_meta_request.c 91.29% <100.00%> (ø)
source/s3_request.c 77.40% <100.00%> (+0.20%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

3 participants