[infinispan-dev] Put issues with newly joining node

[Galder Zamarreño]

On Dec 4, 2012, at 11:30 AM, Dan Berindei <dan.berindei at gmail.com> wrote:

> 
> On Tue, Dec 4, 2012 at 11:32 AM, Mircea Markus <mmarkus at redhat.com> wrote:
> 
> On 4 Dec 2012, at 09:22, Sanne Grinovero wrote:
> 
>> [...]
>> 
>> I don't think the cache should ever be in an illegal state to be used
>> after being started. So Infinispan should not require tests to wait
>> for a "cluster to be formed", I'd rather guarantee that after a cache
>> is started it's usable.
> +1. Unless the test relies/verifies internal state, e.g. locks being acquired, data present in the data container etc.
> 
> 
> It's not just a question of what you want to check, it's also a question of what you don't want to check... I think in general a test should focus on a specific issue, and we know state transfer is always a potential source of (unrelated) failures. So I'd rather have tests that do test state transfer and command forwarding, and tests that avoid state transfer and command forwarding (by waiting for the cluster to form completely).
> 
> I'm pretty sure this is another instance of ISPN-2473, and once we have a fix (and a unit test) for this particular failure, MarshallExternalPojosTest could very well wait for the cluster to form and ignore any state transfer-related issues.
> 
> BTW, I also got an exception yesterday in MarshallExternalPojosTest and I investigated it, but in my case the error was much weirder: two nodes both opened a TCP connection to each other, yet none of them received the forwarded command. I've asked Bela to investigate as well, but he didn't find anything suspicious in JGroups.

Ok, wrt ISPN-2541, I suggest holding off until all other known issues have been solved and see if the issue keeps appearing.

It seems to be a good test for catching these issues (indirectly), so it could be useful to verify :)

> 
> 
> 
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev


--
Galder Zamarreño
galder at redhat.com
twitter.com/galderz

Project Lead, Escalante
http://escalante.io

Engineer, Infinispan
http://infinispan.org