[infinispan-dev] Why no use async listener executor?

[Mircea Markus]

On 6 Oct 2011, at 15:52, Galder Zamarreño wrote:

> 
> On Oct 6, 2011, at 4:04 PM, Mircea Markus wrote:
> 
>> 
>> On 5 Oct 2011, at 16:53, Galder Zamarreño wrote:
>> 
>>> Hey guys,
>>> 
>>> Re: https://issues.jboss.org/browse/ISPN-1396
>>> 
>>> While looking into this, I've discovered that we have been creating executors in cache level components, where we're calling submit from @Listener implementations.
>>> 
>>> For example, BaseStateTransferManagerImpl.ViewChangeListener submits a rehash task and TransactionTable.StaleTransactionCleanup submits a tasks to break locks.
>>> 
>>> Did the people that wrote these listeners (Dan & Mircea respectively) consider defining listeners to be called asynchronously? i.e. @Listener(sync = false)
>> this can work indeed for BaseStateTransferManagerImpl.ViewChangeListener? This seems like a small change - can you modify that in the scope of ISPN-1396 or do you want me to look into it?
> 
> BaseStateTransferManagerImpl.ViewChangeListener? Dan seems to disagree. You meant TransactionTable.StaleTransactionCleanup instead?
yes indeed :-)