only visit nodes in EnumerateChildrenAsync when asked

No idea why this was changed this was introduced in: 08f342e (part of #3598) License: MIT Signed-off-by: Steven Allen <[email protected]>
ipfs · Mar 27, 2018 · 08388e7 · 08388e7
1 parent 3cc4658
commit 08388e7
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/merkledag/merkledag.go b/merkledag/merkledag.go
@@ -319,17 +319,17 @@ func EnumerateChildrenAsync(ctx context.Context, getLinks GetLinks, c *cid.Cid,
 	for i := 0; i < FetchGraphConcurrency; i++ {
 		go func() {
 			for ic := range feed {
-				links, err := getLinks(ctx, ic)
-				if err != nil {
-					errChan <- err
-					return
-				}
-
 				setlk.Lock()
-				unseen := visit(ic)
+				shouldVisit := visit(ic)
 				setlk.Unlock()
 
-				if unseen {
+				if shouldVisit {
+					links, err := getLinks(ctx, ic)
+					if err != nil {
+						errChan <- err
+						return
+					}
+
 					select {
 					case out <- links:
 					case <-fetchersCtx.Done():