Re: [infinispan-dev] Recovery with async cache mode
On 10 May 2013, at 09:55, Manik Surtani wrote:
On 9 May 2013, at 21:44, Mircea Markus <mmarkus(a)redhat.com>
wrote:
>>
>> I think the correct thing to do is to fix the test suite and throw an
>> exception if someone tries to enable recovery with async cache mode.
> Or just raise an WARNING for now, just for backward compatibility.
No, I think we should throw an exception as an invalid configuration. Because, well, it
is invalid. If anyone is using async + recovery, it is important that they realise that
it is invalid and fix their config.
fair enough. At the end of the day people need
to turn on xa-enlistment explicitly so they must be looking for recovery if they do do
that.
@Pedro - just make sure that both recovery is enabled *and* XA resource enlistment is set
to true if throwing an exception. I think that should reduce the number of test that fail
significantly.
Cheers,
--
Mircea Markus
Infinispan lead (www.infinispan.org)