On 11-01-13 2:55 PM, Manik Surtani wrote:
I had a look, still have a few concerns:
* Why do we still have DistributedTaskContext?
We don't need it but it is extra layer of indirection and isolation from
API like EmbeddeCacheManager. DistributedTaskContext is a handle to
obtain mapped cache and other caches, a protection against
EmbeddeCacheManager becoming too complex, being removed and/or similar....
* Factory should probably not be genericised. E.g.,
interface DistributedCallableFactory {
DistributedCallable<K, V, T> createDistributedCallable(Cache c,
EmbeddedCacheManager ecm);
}
but then again that will depend on what you think re: my proposal to make this look like
an ExecutorService (see separate email)
The M/R stuff looks good. :)
Ok let me see.