Skip to content

Commit

Permalink
feature/s3/manager: Fix typo in Downloader (#1304)
Browse files Browse the repository at this point in the history
`DefaultPartBodyMaxRetries` and `PartBodyMaxRetries` comment should
refer to "download" instead of "upload".

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
  • Loading branch information
Juneezee authored Jun 17, 2021
1 parent 1d868df commit cd60ebc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions feature/s3/manager/download.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ const DefaultDownloadPartSize = 1024 * 1024 * 5
// when using Download().
const DefaultDownloadConcurrency = 5

// DefaultPartBodyMaxRetries is the default number of retries to make when a part fails to upload.
// DefaultPartBodyMaxRetries is the default number of retries to make when a part fails to download.
const DefaultPartBodyMaxRetries = 3

type errReadingBody struct {
Expand All @@ -53,7 +53,7 @@ type Downloader struct {
// PartSize is ignored if the Range input parameter is provided.
PartSize int64

// PartBodyMaxRetries is the number of retry attempts to make for failed part uploads
// PartBodyMaxRetries is the number of retry attempts to make for failed part downloads.
PartBodyMaxRetries int

// Logger to send logging messages to
Expand Down

0 comments on commit cd60ebc

Please sign in to comment.