Skip to content

fix(podcast): slightly more robust downloading and concurrency - #433

Merged
sentriz merged 1 commit into
masterfrom
pe-del
Dec 21, 2023
Merged

fix(podcast): slightly more robust downloading and concurrency#433
sentriz merged 1 commit into
masterfrom
pe-del

Conversation

@sentriz

@sentriz sentriz commented Dec 17, 2023

Copy link
Copy Markdown
Owner
  • drive download by a single download "event loop" thread
    • controlled concurrency
    • resume-on-restart downloads
    • suspend gonic shutdown until current download is complete
  • re-use pe filename as download destination to avoid .1 .1.1 .1.1.1 etc unique name
  • update error status in more places
@sentriz
sentriz force-pushed the pe-del branch 8 times, most recently from 4d92124 to c69e063 Compare December 20, 2023 19:13
    - drive download by a single download "event loop" thread
        - controlled concurrency
        - resume-on-restart downloads
        - suspend gonic shutdown until current download is complete
    - re-use pe filename as download destination to avoid .1 .1.1 .1.1.1 etc unique names
    - update error status in more places

fixes #432
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant