<html><head><style type="text/css"><!-- DIV {margin:0px;} --></style></head><body><div style="font-family:arial,helvetica,sans-serif;font-size:10pt"><div>I think that it is absurd to have two different @PreDestroy methods in interceptor class.Because they have same functionality. InvocationContext solely exists in the interceptor specification and it is very logical. Because one wants to call next interceptor in the chain. <br><br>Will TCK tests be corrected?<br><br>Thanks;<br><br>--Gurkan<br></div><div style="font-family: arial,helvetica,sans-serif; font-size: 10pt;"><br><div style="font-family: arial,helvetica,sans-serif; font-size: 10pt;"><font face="Tahoma" size="2"><hr size="1"><b><span style="font-weight: bold;">From:</span></b> Gavin King <gavin.king@gmail.com><br><b><span style="font-weight: bold;">To:</span></b> Marius Bogoevici <mariusb@redhat.com><br><b><span style="font-weight: bold;">Cc:</span></b> Gurkan Erdogdu
<gurkanerdogdu@yahoo.com>; weld-dev@lists.jboss.org<br><b><span style="font-weight: bold;">Sent:</span></b> Mon, November 30, 2009 10:43:42 PM<br><b><span style="font-weight: bold;">Subject:</span></b> Re: [weld-dev] TCK Interceptors Classes<br></font><br>
Well, the problem is that the interceptors spec is in general not very<br>clearly written.<br><br>On Mon, Nov 30, 2009 at 3:42 PM, Gavin King <<a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a>> wrote:<br>> Hrm section 2.6 of the common annotations spec seems to confirm your<br>> interpretation. What's very problematic here is that the interceptors<br>> spec uses a different terminology to talk about the callbacks that it<br>> is defining. Which leads to my interpretation that it is defining a<br>> different, distinct set of callbacks.<br>><br>> On Mon, Nov 30, 2009 at 3:08 PM, Marius Bogoevici <<a ymailto="mailto:mariusb@redhat.com" href="mailto:mariusb@redhat.com">mariusb@redhat.com</a>> wrote:<br>>> Gavin,<br>>><br>>> This is very ambiguous, as the 1.1 version of the Interceptors specification<br>>> states very clearly the signature rules for
defining lifecycle interceptor<br>>> methods on interceptor classes and target classes.<br>>><br>>> Also, this could mean that an interceptor class can specify two different<br>>> @PostConstruct or @PreDestroy methods, which would refer to different<br>>> targets (the intercepted instance/the interceptor itself), but the<br>>> specification says very clearly:<br>>> "At most one method of a given interceptor class can be designated as an<br>>> around-invoke method, an around-timeout method, a post-construct method, or<br>>> pre-destroy method."<br>>><br>>> Also, it is not very clear to me what would be the benefit of a separate<br>>> @PostConstruct/@PreDestroy method for the interceptor itself, as interceptor<br>>> lifecycles are virtually the same as for the target objects.<br>>><br>>> Marius<br>>><br>>><br>>><br>>> Gavin King
wrote:<br>>><br>>> Check section 5.2.5 of the EE spec. It appears to confirm my<br>>> understanding of this stuff.<br>>><br>>> On Mon, Nov 30, 2009 at 2:33 PM, Gavin King <<a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a>> wrote:<br>>><br>>><br>>> At least, that's my understanding of how interceptors are treated in<br>>> EE6. You would have to check with Roberto and Ken for an absolutely<br>>> definitive answer.<br>>><br>>> On Mon, Nov 30, 2009 at 2:32 PM, Gavin King <<a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a>> wrote:<br>>><br>>><br>>> Right, but the interceptor itself has a lifecycle. It's a kind of<br>>> managed bean. So it can have the callbacks that all managed beans can<br>>> have.<br>>><br>>> On Mon, Nov 30, 2009 at
2:17 PM, Gurkan Erdogdu <<a ymailto="mailto:gurkanerdogdu@yahoo.com" href="mailto:gurkanerdogdu@yahoo.com">gurkanerdogdu@yahoo.com</a>><br>>> wrote:<br>>><br>>><br>>> There are two differents scenario for lifecycle callbacks in interceptors<br>>> specification<br>>><br>>> 1* Used in interceptor class with InvocationContext parameter<br>>> @PreDestroy<br>>> public void blabla(InvocationContext){}<br>>> 2* Used in bean class without any parameter<br>>> @PreDestroy<br>>> public void blabla(){}<br>>><br>>> In TCK, @PreDestroy is used in interceptor class. So it may take<br>>> InvocationContext.<br>>><br>>> --Gurkan<br>>><br>>> ________________________________<br>>> From: Gavin King <<a ymailto="mailto:gavin.king@gmail.com"
href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a>><br>>> To: Gurkan Erdogdu <<a ymailto="mailto:gurkanerdogdu@yahoo.com" href="mailto:gurkanerdogdu@yahoo.com">gurkanerdogdu@yahoo.com</a>><br>>> Cc: <a ymailto="mailto:weld-dev@lists.jboss.org" href="mailto:weld-dev@lists.jboss.org">weld-dev@lists.jboss.org</a><br>>> Sent: Mon, November 30, 2009 9:10:17 PM<br>>> Subject: Re: [weld-dev] TCK Interceptors Classes<br>>><br>>> Hrm, I think there are two kinds of @PreDestroy methods for an interceptor:<br>>><br>>> @PreDestroy void foo(InvocationContext) { .. } -> the intercepted<br>>> bean is being destroyed<br>>> @PreDestroy void foo() { .. } -> the interceptor itself is being destroyed<br>>><br>>> Right?<br>>><br>>> On Mon, Nov 30, 2009 at 1:34 PM, Gurkan Erdogdu <<a ymailto="mailto:gurkanerdogdu@yahoo.com"
href="mailto:gurkanerdogdu@yahoo.com">gurkanerdogdu@yahoo.com</a>><br>>> wrote:<br>>><br>>><br>>> Hi;<br>>><br>>> Some interceptors classes in the TCK test suites implement @PreDestroy<br>>> methods. AFAIK, interceptors specification says that methods with<br>>> @PreDestroy in interceptor class must take InvocationContext parameter.<br>>> But<br>>> in TCK, those methods do not take InvocationContext parameter<br>>><br>>> For example:<br>>><br>>> org.jboss.jsr299.tck.tests.context.dependent.TransactionalInterceptor<br>>><br>>> @PreDestroy public void destroy()<br>>> {<br>>> destroyed = true;<br>>> }<br>>><br>>> Is it correct?<br>>><br>>> --Gurkan<br>>><br>>><br>>><br>>><br>>>
_______________________________________________<br>>> weld-dev mailing list<br>>> <a ymailto="mailto:weld-dev@lists.jboss.org" href="mailto:weld-dev@lists.jboss.org">weld-dev@lists.jboss.org</a><br>>> <a href="https://lists.jboss.org/mailman/listinfo/weld-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/weld-dev</a><br>>><br>>><br>>><br>>> --<br>>> Gavin King<br>>> <a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a><br><span>>> <a target="_blank" href="http://in.relation.to/Bloggers/Gavin">http://in.relation.to/Bloggers/Gavin</a></span><br><span>>> <a target="_blank" href="http://hibernate.org">http://hibernate.org</a></span><br><span>>> <a target="_blank" href="http://seamframework.org">http://seamframework.org</a></span><br>>><br>>><br>>><br>>><br>>> --<br>>> Gavin King<br>>> <a
ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a><br>>> <a href="http://in.relation.to/Bloggers/Gavin" target="_blank">http://in.relation.to/Bloggers/Gavin</a><br>>> <a href="http://hibernate.org" target="_blank">http://hibernate.org</a><br>>> <a href="http://seamframework.org" target="_blank">http://seamframework.org</a><br>>><br>>><br>>><br>>> --<br>>> Gavin King<br>>> <a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a><br>>> <a href="http://in.relation.to/Bloggers/Gavin" target="_blank">http://in.relation.to/Bloggers/Gavin</a><br>>> <a href="http://hibernate.org" target="_blank">http://hibernate.org</a><br>>> <a href="http://seamframework.org" target="_blank">http://seamframework.org</a><br>>><br>>><br>>><br>>><br>>><br>><br>><br>><br>>
--<br>> Gavin King<br>> <a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a><br>> <a href="http://in.relation.to/Bloggers/Gavin" target="_blank">http://in.relation.to/Bloggers/Gavin</a><br>> <a href="http://hibernate.org" target="_blank">http://hibernate.org</a><br>> <a href="http://seamframework.org" target="_blank">http://seamframework.org</a><br>><br><br><br><br>-- <br>Gavin King<br><a ymailto="mailto:gavin.king@gmail.com" href="mailto:gavin.king@gmail.com">gavin.king@gmail.com</a><br><a href="http://in.relation.to/Bloggers/Gavin" target="_blank">http://in.relation.to/Bloggers/Gavin</a><br><a href="http://hibernate.org" target="_blank">http://hibernate.org</a><br><a href="http://seamframework.org" target="_blank">http://seamframework.org</a><br></div></div>
<!-- cg11.c1.mail.mud.yahoo.com compressed/chunked Wed Nov 25 10:02:22 PST 2009 -->
</div><br>
</body></html>