Merge pull request #183 from molpako/fix-comment

k8s-ci-robot · web-flow · commit 40838cb6af54 · 2025-10-24T08:03:38.000-07:00
fix: correct chunk size comment to 1 MiB
diff --git a/pkg/iterator/emitters.go b/pkg/iterator/emitters.go
@@ -99,13 +99,13 @@ type VerifierEmitter struct {
 	TargetDevice *os.File
 }
 
-const chunkSize = int64(1024 * 1024) // 4KB
+const chunkSize = int64(1024 * 1024) // 1 MiB
 var (
 	ErrFailedToSeek       = errors.New("failed to seek")
 	ErrContentsDoNotMatch = errors.New("source and target device contents do not match")
 )
 
-// copyBlock will copy chunks of 256 bytes at a time upto to sizeBytes from the source device to the target device.
+// copyBlock will copy chunks of 1 MiB at a time upto to sizeBytes from the source device to the target device.
 func (verifierEmitter *VerifierEmitter) copyBlock(bmd *api.BlockMetadata) error {
 	// Seek to the byteOffset of the source device.
 	_, err := verifierEmitter.SourceDevice.Seek(bmd.ByteOffset, io.SeekStart)

Original file line number	Diff line number	Diff line change
`@@ -99,13 +99,13 @@ type VerifierEmitter struct {`
`99`	`99`	`TargetDevice *os.File`
`100`	`100`	`}`
`101`	`101`
`102`		`-const chunkSize = int64(1024 * 1024) // 4KB`
	`102`	`+const chunkSize = int64(1024 * 1024) // 1 MiB`
`103`	`103`	`var (`
`104`	`104`	`ErrFailedToSeek = errors.New("failed to seek")`
`105`	`105`	`ErrContentsDoNotMatch = errors.New("source and target device contents do not match")`
`106`	`106`	`)`
`107`	`107`
`108`		`-// copyBlock will copy chunks of 256 bytes at a time upto to sizeBytes from the source device to the target device.`
	`108`	`+// copyBlock will copy chunks of 1 MiB at a time upto to sizeBytes from the source device to the target device.`
`109`	`109`	`func (verifierEmitter VerifierEmitter) copyBlock(bmd api.BlockMetadata) error {`
`110`	`110`	`// Seek to the byteOffset of the source device.`
`111`	`111`	`_, err := verifierEmitter.SourceDevice.Seek(bmd.ByteOffset, io.SeekStart)`