add comment

andimarek · andimarek · commit 66f22bf36549 · 2017-08-21T07:37:21.000+02:00
diff --git a/src/main/java/graphql/execution/Async.java b/src/main/java/graphql/execution/Async.java
@@ -46,6 +46,7 @@ public static <T, U> CompletableFuture<List<U>> each(Iterable<T> list, BiFunctio
                 Assert.assertNotNull(cf, "cfFactory must return a non null value");
             } catch (Exception e) {
                 cf = new CompletableFuture<>();
+                // Async.each makes sure that it is not a CompletionException inside a CompletionException
                 cf.completeExceptionally(new CompletionException(e));
             }
             futures.add(cf);

Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,7 @@ public static <T, U> CompletableFuture<List<U>> each(Iterable<T> list, BiFunctio`
`46`	`46`	`Assert.assertNotNull(cf, "cfFactory must return a non null value");`
`47`	`47`	`} catch (Exception e) {`
`48`	`48`	`cf = new CompletableFuture<>();`
	`49`	`+ // Async.each makes sure that it is not a CompletionException inside a CompletionException`
`49`	`50`	`cf.completeExceptionally(new CompletionException(e));`
`50`	`51`	`}`
`51`	`52`	`futures.add(cf);`