[infinispan-dev] Command cancellation

[Vladimir Blagojevic]

On 12-10-11 7:45 AM, Manik Surtani wrote:
>
> On 1 Oct 2012, at 14:40, Vladimir Blagojevic <vblagoje at redhat.com 
> <mailto:vblagoje at redhat.com>> wrote:
>
>> Now that I am actually implementing this I am wondering why don't we 
>> simply add getUUID to CacheRpcCommand. CancellationService will use 
>> this UUID for command cancellation, some other services might need 
>> UUID for something else. Introducing CancellableCommand does not add 
>> much except bunch of instanceof constructs and explicitly connects 
>> UUID with concept of command cancellation which might not been only 
>> valid use of command UUID.
>
> No, this should be on CancellableCommand only.  No sense in 
> engineering for something we may not need and polluting interfaces.
>
Ok, no problem. I'll adjust the PR. However, I need to mark that only 
CacheRpcCommand can be cancellable. So we make CancellableCommand extend 
CacheRpcCommand?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/infinispan-dev/attachments/20121011/a7200107/attachment.html