linkedin / ambry

Distributed object store
https://github.com/linkedin/ambry/wiki
Apache License 2.0
1.75k stars 275 forks source link

Support 100-continue PUT request #2762

Closed SophieGuo410 closed 6 months ago

SophieGuo410 commented 6 months ago

This pr supported the Expect:100-continue feature. This header indicates that the client is asking the server to confirm if it is acceptable to send the request body before actually sending it. The server, upon receiving such a request with the "Expect: 100-continue" header, can respond with a "100 Continue" status code if it is ready to receive the request body. If not ready or if there's an issue, it can respond with an appropriate error code.

codecov-commenter commented 6 months ago

Codecov Report

Attention: Patch coverage is 26.66667% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 18.60%. Comparing base (b4f759e) to head (7059594). Report is 4 commits behind head on master.

Files Patch % Lines
...com/github/ambry/frontend/NamedBlobPutHandler.java 33.33% 3 Missing and 1 partial :warning:
...a/com/github/ambry/rest/NettyMessageProcessor.java 0.00% 2 Missing and 2 partials :warning:
...va/com/github/ambry/rest/NettyResponseChannel.java 25.00% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2762 +/- ## ============================================= - Coverage 70.43% 18.60% -51.84% + Complexity 11597 2871 -8726 ============================================= Files 837 837 Lines 71246 71258 +12 Branches 8552 8556 +4 ============================================= - Hits 50184 13257 -36927 - Misses 18432 56774 +38342 + Partials 2630 1227 -1403 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.