Merge pull request #4664 from deitch/ctr-client-optimize

mxpv · web-flow · commit 619f96c757e7 · 2020-10-27T10:13:06.000-07:00
use proper buffer size for content get
diff --git a/cmd/ctr/commands/content/content.go b/cmd/ctr/commands/content/content.go
@@ -78,7 +78,9 @@ var (
 			}
 			defer ra.Close()
 
-			_, err = io.Copy(os.Stdout, content.NewReader(ra))
+			// use 1MB buffer like we do for ingesting
+			buf := make([]byte, 1<<20)
+			_, err = io.CopyBuffer(os.Stdout, content.NewReader(ra), buf)
 			return err
 		},
 	}

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,9 @@ var (`
`78`	`78`	`}`
`79`	`79`	`defer ra.Close()`
`80`	`80`
`81`		`- _, err = io.Copy(os.Stdout, content.NewReader(ra))`
	`81`	`+ // use 1MB buffer like we do for ingesting`
	`82`	`+ buf := make([]byte, 1<<20)`
	`83`	`+ _, err = io.CopyBuffer(os.Stdout, content.NewReader(ra), buf)`
`82`	`84`	`return err`
`83`	`85`	`},`
`84`	`86`	`}`