search

awslabs / aws-c-s3

C99 library implementation for communicating with the S3 service, designed for maximizing throughput on high bandwidth EC2 instances.
Apache License 2.0
94 stars 38 forks source link

Mem limiter validation #385

Closed DmitriyMusatkin closed 9 months ago

DmitriyMusatkin commented 9 months ago

Issue #, if available:

Description of changes: relax some cases of validation in mem limiter

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

codecov-commenter commented 9 months ago

Codecov Report

Merging #385 (a76041f) into main (35eee82) will increase coverage by 0.12%. The diff coverage is 71.42%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/awslabs/aws-c-s3/pull/385/graphs/tree.svg?width=650&height=150&src=pr&token=J4KP54FVLF&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs)](https://app.codecov.io/gh/awslabs/aws-c-s3/pull/385?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs) ```diff @@ Coverage Diff @@ ## main #385 +/- ## ========================================== + Coverage 88.45% 88.58% +0.12% ========================================== Files 21 21 Lines 5941 5938 -3 ========================================== + Hits 5255 5260 +5 + Misses 686 678 -8 ``` | [Files](https://app.codecov.io/gh/awslabs/aws-c-s3/pull/385?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs) | Coverage Δ | | |---|---|---| | [source/s3.c](https://app.codecov.io/gh/awslabs/aws-c-s3/pull/385?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c291cmNlL3MzLmM=) | `95.65% <ø> (ø)` | | | [source/s3\_buffer\_pool.c](https://app.codecov.io/gh/awslabs/aws-c-s3/pull/385?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c291cmNlL3MzX2J1ZmZlcl9wb29sLmM=) | `96.79% <83.33%> (+4.97%)` | :arrow_up: | | [source/s3\_client.c](https://app.codecov.io/gh/awslabs/aws-c-s3/pull/385?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c291cmNlL3MzX2NsaWVudC5j) | `87.95% <0.00%> (ø)` | |