Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add set_exception for CRT Transfer future #195

Merged
merged 1 commit into from Apr 20, 2021

Conversation

kyleknap
Copy link
Member

This enables more parity with the threaded, Python transfer manager where a subscriber can set an exception in the on_done
callback such as failing to delete the source file after transferring it.

This enables more parity with the threaded, Python transfer
manager where a subscriber can set an exception in the on_done
callback such as failing to delete the source file after
transfering it.
@codecov-commenter
Copy link

Codecov Report

Merging #195 (b7c1df5) into develop (5802ec3) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #195      +/-   ##
===========================================
- Coverage    86.13%   86.01%   -0.12%     
===========================================
  Files           16       16              
  Lines         2762     2767       +5     
===========================================
+ Hits          2379     2380       +1     
- Misses         383      387       +4     
Impacted Files Coverage Δ
s3transfer/crt.py 0.00% <0.00%> (ø)
s3transfer/tasks.py 96.66% <0.00%> (+0.83%) ⬆️

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 5802ec3...b7c1df5. Read the comment docs.

@kyleknap kyleknap merged commit f19bf5f into boto:develop Apr 20, 2021
@kyleknap kyleknap deleted the crt-set-exception branch April 20, 2021 17:42
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.

None yet

3 participants