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

Kadai3-2 by kaznishi #53

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Kadai3-2 by kaznishi #53

wants to merge 3 commits into from

Conversation

kaznishi
Copy link

@kaznishi kaznishi commented Jul 16, 2018

(キャンセル処理を入れるのに苦戦しました。力尽きたのでテストは省略しました…。)

  • 分割ダウンロードを行う
    • Rangeアクセスを用いる
    • いくつかのゴルーチンでダウンロードしてマージする
    • エラー処理を工夫する
      • errgroupを使って並行で走るダウンロードのエラーがまとまるようにしてみました
    • キャンセルが発生した場合の実装を行う
      • Ctrl + C等によるシグナルでのキャンセルが発生した場合に、すぐにプロセスを終了せずに、中途半端になっているダウンロードファイル群を掃除してから終了するようにしました。

@kaznishi kaznishi added the kadai3 This issue or pull request already exists label Jul 16, 2018
@kaznishi kaznishi self-assigned this Jul 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kadai3 This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant