<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<html>
<body link="#355491" alink="#4262a1" vlink="#355491" style="background: #e2e2e2; margin: 0; padding: 20px;">
<div>
<table cellpadding="0" bgcolor="#FFFFFF" border="0" cellspacing="0" style="border: 1px solid #dadada; margin-bottom: 30px; width: 100%; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tbody>
<tr>
<td>
<table border="0" cellpadding="0" cellspacing="0" bgcolor="#FFFFFF" style="border: solid 2px #ccc; background: #dadada; width: 100%; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tbody>
<tr>
<td bgcolor="#000000" valign="middle" height="58px" style="border-bottom: 1px solid #ccc; padding: 20px; -moz-border-radius-topleft: 3px; -moz-border-radius-topright: 3px; -webkit-border-top-right-radius: 5px; -webkit-border-top-left-radius: 5px;">
<h1 style="color: #333333; font: bold 22px Arial, Helvetica, sans-serif; margin: 0; display: block !important;">
<!-- To have a header image/logo replace the name below with your img tag -->
<!-- Email clients will render the images when the message is read so any image -->
<!-- must be made available on a public server, so that all recipients can load the image. -->
<a href="http://community.jboss.org/index.jspa" style="text-decoration: none; color: #E1E1E1">JBoss Community</a></h1>
</td>
</tr>
<tr>
<td bgcolor="#FFFFFF" style="font: normal 12px Arial, Helvetica, sans-serif; color:#333333; padding: 20px; -moz-border-radius-bottomleft: 4px; -moz-border-radius-bottomright: 4px; -webkit-border-bottom-right-radius: 5px; -webkit-border-bottom-left-radius: 5px;"><h3 style="margin: 10px 0 5px; font-size: 17px; font-weight: normal;">
Transaction problems, possible bug in ApplicationExceptionComponentMetaDataLoaderFactory
</h3>
<span style="margin-bottom: 10px;">
created by <a href="http://community.jboss.org/people/benoit.heinrich">Benoit Heinrich</a> in <i>EJB3</i> - <a href="http://community.jboss.org/message/599161#599161">View the full discussion</a>
</span>
<hr style="margin: 20px 0; border: none; background-color: #dadada; height: 1px;">
<div class="jive-rendered-content"><p>Hi All,</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>I think I've found a possible bug with transaction attributes management in jboss 6.0.0 Final.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>The problem appears when using the @TransactionAttribute annotation on an EJB bean with the @Interceptors.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>When using that it seems that the @TransactionAttribute annotation is ignored.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>By removing the @Interceptors annotation it works and the right transaction attribute is used.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>After doing some debugging, I think I've finally found the problem in jboss-ejb3-transactions module in the ApplicationExceptionComponentMetaDataLoaderFactory class.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>I think the bug is around the createComponentMetaDataRetrieval() method where it searches for the <strong>ApplicationExceptionMetaData appExMetaData</strong>, but then it checks if the <strong>metaData</strong> exists instead of checking for the <strong>appExMetaData</strong>.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><pre class="jive-pre"><code class="jive-code jive-java">   <font color="navy"><b>public</b></font> MetaDataRetrieval createComponentMetaDataRetrieval(JBossEnterpriseBeanMetaData metaData, Signature signature,
         ScopeKey key, ClassLoader classLoader)
   <font color="navy">{</font>
      <font color="navy"><b>if</b></font>(signature <font color="navy"><b>instanceof</b></font> ClassSignature)
      <font color="navy">{</font>
         ApplicationExceptionMetaData appExMetaData = findApplicationException(metaData, signature.getName());
         <font color="navy"><b>if</b></font>(metaData != <font color="navy"><b>null</b></font>)
            <font color="navy"><b>return</b></font> <font color="navy"><b>new</b></font> BridgedMetaDataLoader<ApplicationExceptionMetaData>(key, appExMetaData, classLoader, defaultBridges);
      <font color="navy">}</font>
      <font color="navy"><b>return</b></font> <font color="navy"><b>null</b></font>;
   <font color="navy">}</font>
 
</code></pre><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>So then, as the metaData here is always no-null, it then always fallback to a this MetaDataRetrieval instance which in the end doesn't read the TransactionAttribute properly and so, ignores the attribute type.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>I'm wondering what other impacts this could have.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>If someone can verify that this is the actual problem, I'll be happy to test the bug fix.</p><p>Meanwhile I'll get latest from svn and make the change locally and try it.  So I can also confirm that it works.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>btw - I've not been able to reproduce with a simple test case and this is happening in a very large project that I can't easily extract the code from.</p><p>fyi - I'm using jboss-seam 2.2.1.Final in this project, and I'm wondering if this can also be a lead for reproducing that problem.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>Let me know if I should open a Jira for that.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>Cheers,</p><p>/Benoit</p></div>
<div style="background-color: #f4f4f4; padding: 10px; margin-top: 20px;">
<p style="margin: 0;">Reply to this message by <a href="http://community.jboss.org/message/599161#599161">going to Community</a></p>
<p style="margin: 0;">Start a new discussion in EJB3 at <a href="http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2029">Community</a></p>
</div></td>
</tr>
</tbody>
</table>
</td>
</tr>
</tbody>
</table>
</div>
</body>
</html>