Re: [infinispan-dev] CDI support

2011-07-14 Thread Manik Surtani
+1, I think so too. If you have the time, could you go ahead and merge it in to master? Some docs with examples + a blog about it would be awesome too. :) I presume this is something Kevin would do? On 13 Jul 2011, at 18:36, Sanne Grinovero wrote: Since you're the expert, if you like it

Re: [infinispan-dev] CDI support

2011-07-14 Thread Pete Muir
On 14 Jul 2011, at 10:36, Manik Surtani wrote: +1, I think so too. If you have the time, could you go ahead and merge it in to master? Some docs with examples + a blog about it would be awesome too. :) I presume this is something Kevin would do? I'll work with Kevin on this :-)

Re: [infinispan-dev] CDI support

2011-07-14 Thread Manik Surtani
On 14 Jul 2011, at 10:47, Pete Muir wrote: On 14 Jul 2011, at 10:36, Manik Surtani wrote: +1, I think so too. If you have the time, could you go ahead and merge it in to master? Some docs with examples + a blog about it would be awesome too. :) I presume this is something Kevin

Re: [infinispan-dev] CDI support

2011-07-14 Thread Pete Muir
On 14 Jul 2011, at 11:35, Manik Surtani wrote: On 14 Jul 2011, at 10:47, Pete Muir wrote: On 14 Jul 2011, at 10:36, Manik Surtani wrote: +1, I think so too. If you have the time, could you go ahead and merge it in to master? Some docs with examples + a blog about it would be

[infinispan-dev] For Eclipse users: setup the annotation processors

2011-07-14 Thread Sanne Grinovero
Yesterday during one of the changes I made about dependencies I removed the committed eclipse project file which setup the annotation processors for Eclipse, mainly because it was not valid anymore with the current dependencies. Fred Bricon from the JBoss Tools team is working on

Re: [infinispan-dev] For Eclipse users: setup the annotation processors

2011-07-14 Thread Pete Muir
On 14 Jul 2011, at 12:34, Sanne Grinovero wrote: Yesterday during one of the changes I made about dependencies I removed the committed eclipse project file which setup the annotation processors for Eclipse, mainly because it was not valid anymore with the current dependencies. Haha, I only

[infinispan-dev] IllegalStateException when receiving invalidation messages on a stopped/stopping cache

2011-07-14 Thread Sanne Grinovero
Hello, I was looking into the many stacktraces being thrown from the testsuite, this one caught my attention (see below). Can't we just discard such errors? if the cache is stopping, or stopped already, we shouldn't really care for invalidations - especially stacktraces and exceptions being

Re: [infinispan-dev] IllegalStateException when receiving invalidation messages on a stopped/stopping cache

2011-07-14 Thread Manik Surtani
Patch looks good. Go ahead and issue a pull req. On 14 Jul 2011, at 16:26, Sanne Grinovero wrote: Hello, I was looking into the many stacktraces being thrown from the testsuite, this one caught my attention (see below). Can't we just discard such errors? if the cache is stopping, or

Re: [infinispan-dev] IllegalStateException when receiving invalidation messages on a stopped/stopping cache

2011-07-14 Thread Dan Berindei
TBH I'm not sure why the patch helped with the error messages in the log, I always thought that doing this would just replace the IllegalStateException with an InterruptedException (since do some more stuff after invalidation). Of course the best solution would be to not do rehashing at all when