Skip to content

Support presigning S3 createMultipartUpload, uploadPart, completeMultipartUpload and abortMultipartUpload requests.#1646

Merged
millems merged 3 commits into
masterfrom
millem/presign-part-operations
Feb 13, 2020
Merged

Support presigning S3 createMultipartUpload, uploadPart, completeMultipartUpload and abortMultipartUpload requests.#1646
millems merged 3 commits into
masterfrom
millem/presign-part-operations

Conversation

@millems

@millems millems commented Feb 12, 2020

Copy link
Copy Markdown
Contributor

Requested in #203 (comment)

…ipartUpload and abortMultipartUpload requests.

Requested in #203 (comment)
@millems millems mentioned this pull request Feb 12, 2020
@codecov-io

codecov-io commented Feb 12, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1646 into master will decrease coverage by 0.12%.
The diff coverage is 77.39%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1646      +/-   ##
============================================
- Coverage     74.44%   74.32%   -0.13%     
- Complexity      741      762      +21     
============================================
  Files           945      948       +3     
  Lines         29621    29770     +149     
  Branches       2332     2332              
============================================
+ Hits          22051    22126      +75     
- Misses         6498     6573      +75     
+ Partials       1072     1071       -1
Flag Coverage Δ Complexity Δ
#unittests 74.32% <77.39%> (-0.13%) 762 <78> (+21)
Impacted Files Coverage Δ Complexity Δ
...s/dynamodb/mappingclient/model/ConditionCheck.java 8.52% <ø> (ø) 2 <0> (?)
...appingclient/model/CreateTableEnhancedRequest.java 51.28% <ø> (ø) 7 <0> (ø) ⬇️
.../mappingclient/core/DefaultDynamoDbAsyncIndex.java 37.83% <ø> (ø) 7 <0> (ø) ⬇️
...amodb/mappingclient/model/LocalSecondaryIndex.java 12.74% <ø> (ø) 4 <0> (ø) ⬇️
...dynamodb/mappingclient/model/BatchWriteResult.java 100% <ø> (ø) 10 <0> (ø) ⬇️
...ingclient/model/BatchWriteItemEnhancedRequest.java 56.52% <ø> (+8.69%) 3 <0> (ø) ⬇️
...amodb/mappingclient/core/DefaultDynamoDbIndex.java 37.83% <ø> (ø) 7 <0> (ø) ⬇️
...namodb/mappingclient/model/BatchGetResultPage.java 100% <ø> (ø) 4 <0> (ø) ⬇️
...odb/mappingclient/model/TransactGetResultPage.java 60% <ø> (ø) 4 <0> (ø) ⬇️
...modb/mappingclient/model/QueryEnhancedRequest.java 50% <ø> (ø) 9 <0> (ø) ⬇️
... and 39 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ceddc7...4402aed. Read the comment docs.

@millems millems requested a review from dagnir February 13, 2020 20:15
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 24 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@millems millems merged commit 5d3b841 into master Feb 13, 2020
@millems millems deleted the millem/presign-part-operations branch February 13, 2020 23:39
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