Skip to content

Fix Overdrive

Peter-Jan Brone requested to merge pj/fix-overdrive into pj/launched-worker-info

MERGE REQUEST

MR Guidelines

Overview

This MR fixes a couple of issues with the overdrive code, resulting in download failures that were reported by the community.

There were three main issues causing this to happen:

  • a faulty break statement, breaking out of an outer for loop instead of the inner loop
  • adjustedReadDuration was not taking into account a potential cooldown of the worker's read queue
  • an exponential backoff mechanism maxed out at 30s

This MR also updates the minimumLookahead from 2MiB to 8MiB, improving DL speeds 3x on siasky.dev.

Example for Visual Changes

N/A

Checklist

Review and complete the checklist to ensure that the MR is complete before assigned to an approver.

  • All new methods or updated methods have clear docstrings
  • Testing added or updated for new methods
  • Any new packages are added to Makefile and .gitlab-ci.yml
  • API documentation updated for API updates
  • Module README.md updated for changes to workflow
  • Issue added to Sia-UI repo for new supporting features
  • Changelog File Created

Issues Closed

N/A

Merge request reports