21891: Comment to explain mysterious numbers 13 and 17 in test case.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

diff --git a/lib/crunchrun/copier_test.go b/lib/crunchrun/copier_test.go
index 3348a879a7fd98867aef3ba7dd46999089c9c750..9ae71b9a22ced61a948f4ad90117b0ba0f78442b 100644 (file)
--- a/lib/crunchrun/copier_test.go
+++ b/lib/crunchrun/copier_test.go
@@ -348,6 +348,10 @@ func (s *copierSuite) testCopyFromLargeCollection(c *check.C, writable bool) {
        c.Log(s.log.String())
 
        // Check some files to ensure they were copied properly.
+       // Specifically, arbitrarily check every 17th file in every
+       // 13th dir.  (This is better than checking all of the files
+       // only in that it's less likely to show up as a distracting
+       // signal in CPU profiling.)
        for i := 0; i < 100; i += 13 {
                for j := 0; j < 100; j += 17 {
                        fnm := fmt.Sprintf("/fakecollection/dir%d/dir%d/file%d", i, j, j)