<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title></title>
</head>
<body>
<div name="messageReplySection" style="font-size: 14px; font-family: -apple-system, BlinkMacSystemFont, sans-serif;"><br />
On 8 juin 2017 à 09:23 +0200, Martin Kouba <mkouba@redhat.com>, wrote:<br />
<blockquote type="cite" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #1abc9c;"><br />
Dne 7.6.2017 v 14:21 Antoine Sabot-Durand napsal(a):<br />
<blockquote type="cite" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #e67e22;">Hi all,<br />
<br />
It seems that we introduced a mistake in CDI 2.0 regarding its alignment<br />
with JTA. See https://issues.jboss.org/browse/CDI-705<br />
<br />
Transactional observer bound to BEFORE_COMPLETION shouldn’t be triggered<br />
if the transaction is rolled back or marked for roll back.<br />
<br />
I suggest that we skip the matching test in the TCK [1] and allow impl<br />
to skip this rule and align on JTA.<br /></blockquote>
<br />
+1<br />
<br />
<blockquote type="cite" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #e67e22;"><br />
JTA should be corrected in a coming MR.<br /></blockquote>
<br />
I suppose you mean "CDI should be corrected..." ;-) <br /></blockquote>
<div><br /></div>
<div>Yes, Of course, CDI should be corrected ;)</div>
<div><br /></div>
<blockquote type="cite" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #1abc9c;"><br />
<blockquote type="cite" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #e67e22;"><br />
Thanks for your feedback<br />
<br />
<br />
<br />
Antoine<br />
<br />
<br />
<br />
<br />
[1] https://github.com/cdi-spec/cdi-tck/blob/master/impl/src/main/java/org/jboss/cdi/tck/tests/event/observer/transactional/roolback/TransactionalObserverRollbackTest.java<br />
<br />
<br />
<br />
_______________________________________________<br />
cdi-dev mailing list<br />
cdi-dev@lists.jboss.org<br />
https://lists.jboss.org/mailman/listinfo/cdi-dev<br />
<br />
Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.<br />
<br /></blockquote>
<br />
--<br />
Martin Kouba<br />
Senior Software Engineer<br />
Red Hat, Czech Republic<br /></blockquote>
</div>
</body>
</html>