[infinispan-dev] Transaction table cleanup

[Manik Surtani]

On 19 Oct 2012, at 02:44, Mircea Markus <mircea.markus at jboss.com> wrote:

> 
> On 18 Oct 2012, at 18:21, Manik Surtani wrote:
> 
>> On 19 Oct 2012, at 02:19, Mircea Markus <mircea.markus at jboss.com> wrote:
>> 
>>> 
>>> On 18 Oct 2012, at 16:41, Manik Surtani wrote:
>>> 
>>>> 
>>>> On 18 Oct 2012, at 17:45, Vladimir Blagojevic <vblagoje at redhat.com> wrote:
>>>> 
>>>>> Hey guys,
>>>>> 
>>>>> Investigating why EmbeddedCacheManager#cacheRemove hiccups 
>>>>> DistributedTwoNodesMapReduceTest. As you might recall upon end of 
>>>>> MapReduceTask there is EmbeddedCacheManager#cacheRemove call to remove 
>>>>> intermediate caches across the cluster.
>>> I think you meant ECM#removeCache?
>>>>> Very often, almost every test 
>>>>> run, execution of cache remove is blocked by ongoing transactions that 
>>>>> have not completed - more specifically TransactionTable shows pending 
>>>>> remote transaction. This in turn prevents cache stop call, which in turn 
>>>>> causes a timeout on cache remove thus failing the test.
>>>>> 
>>>>> MapReduce uses txs in MapReduceManagerImpl#combine - it might well be 
>>>>> that there is smth wrong there or it could be that somehow 
>>>> 
>>>> ^^^ Does this mean that to use Map/Reduce now, we *require* that a transaction manager is available, configured and present?
>>> Not really. It's only that the problem Vladimir describes only occurs when you run the MR against transactional caches (there is a tx leak which prevents caches to stop, that should be fixed by a pull request I sent).
>> 
>> Ok.  I was just wondering, since Vladimir did say MR uses txs in MapReduceManagerImpl#combine.  Vladimir, could you pls clarify?
> You're right actually, the temporary cache created is transactional. it is built in the CreateCacheCommand and relies on the DummyTransactionManager, might be better to use batching perhaps? Or even not require for this cache to be transactional?

Yes, we should keep such internal caches as simple as possible.

--
Manik Surtani
manik at jboss.org
twitter.com/maniksurtani

Platform Architect, JBoss Data Grid
http://red.ht/data-grid

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/infinispan-dev/attachments/20121019/e2d1fdeb/attachment.html