<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 12-10-29 8:49 PM, Manik Surtani
wrote:<br>
</div>
<blockquote
cite="mid:CD25F620-E197-4239-B481-65BB40E33B72@jboss.org"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<br>
<div>
<div>On 19 Oct 2012, at 17:40, Mircea Markus <<a
moz-do-not-send="true" href="mailto:mircea.markus@jboss.com">mircea.markus@jboss.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<div style="word-wrap: break-word; -webkit-nbsp-mode: space;
-webkit-line-break: after-white-space; "><br>
<div>
<div>On 19 Oct 2012, at 07:41, Vladimir Blagojevic wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 12-10-19 4:15 AM,
Manik Surtani wrote:<br>
</div>
<blockquote
cite="mid:6B25062A-6798-4852-8C40-75BCAF17B19F@jboss.org"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<br>
<div>
<div>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?</div>
<div><br>
</div>
<div>Yes, we should keep such internal caches as
simple as possible.</div>
<br>
</div>
</blockquote>
Mircea/Manik, it could be batch I believe. If you
recall Mircea, you and I talked about how to
effectively move data across cluster deadlock free -
we even agreed we should blog post about it :-) <a
moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://issues.jboss.org/browse/ISPN-2156">https://issues.jboss.org/browse/ISPN-2156</a><br>
<br>
Mircea/Manik could I get some advice and code review
for MapReduceManagerImpl#combine?<br>
</div>
</blockquote>
<div>Thanks Vladimir. The only change to make
is CreateCacheCommand.create, configure the cache to be
batchable[1]. Then in the MapReduceManagerImpl#combine,
don't use the TransactionManager.beggin/commit but the
batch API.</div>
</div>
</div>
</blockquote>
<div><br>
</div>
<div>Is this in? Have we redone M/R not to rely on transactions
and instead to use batching yet? Vladimir?</div>
<br>
</div>
</blockquote>
Manik, not yet. I am done with cancellation now and I am moving onto
this one. I've tried it over the weekend and it still deadlocks from
time to time as described here
<a class="moz-txt-link-freetext" href="https://issues.jboss.org/browse/ISPN-2439">https://issues.jboss.org/browse/ISPN-2439</a><br>
<br>
This was not the case before (no changes whatsoever to m/r codebase)
and I requested Mircea's help here. We'll convert it to batches and
see to make sure these deadlocks do not occur. <br>
<br>
Vladimir<br>
</body>
</html>