1. Also I just noticed this now, but do we really need to make a copy here?

2. And also why are we adding more elements to this collection at [1] ?  Are you thinking the task would add to the collection and then able to handle additional values?  If so this seems very brittle as it matters in what order they are added since if the element was tested to be valid before it was added it wouldn't work.

[1] https://github.com/mmarkus/infinispan/blob/16aca02a8c592e0546a2406dcfc03b606ee05efe/core/src/main/java/org/infinispan/interceptors/CacheLoaderInterceptor.java#L193