mirrors/git-annex
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

comment

Browse source
This commit is contained in:
Joey Hess 2020-06-03 12:00:44 -04:00
parent 697ebba78f
commit 81a811d935
No known key found for this signature in database
GPG key ID: DB12DB0FF05F8F38
1 changed files with 19 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

19
doc/todo/more_extensive_retries_to_mask_transient_failures/comment_10_f39b7dad4fa1bf4a70bde2e74372fe00._comment Normal file
View file

@ -0,0 +1,19 @@
[[!comment format=mdwn
username="joey"
subject="""comment 10"""
date="2020-06-03T15:51:41Z"
content="""
Just using async is not enough, if thread A asyncs thread B which asyncs
thread C, and then A cancels B, that leaves C running.
withAsync avoids that problem. So around 30 calls to `async` need to be
converted. Also, any library that might use `async` would be a problem.
(Seems that `concurrently` and `race` also avoid the problem, being built on
withAsync.)
(Some uses of `async` do avoid the problem because the async thread exits
for some other reason when the outer thread is terminated. Eg,
Database.Handle auto-terminates its worker thread when the handle gets
garbage collected.)
"""]]