<p dir="ltr">This mainly implies failure cases so I guess as much apps rely on it that apps relying on impl and vendor specific behavior.</p>
<p dir="ltr">That said not sure we need to spend time on it being said EJB are not being updated much and are slowly replaced by other spec.</p>
<p dir="ltr">Side note: this last point makes me think for future integration CDI should limit itself to the "contract" with other spec fully ignoring impl in its own wording. Maybe it is the lesson we need to learn.<br>
</p>
<div class="gmail_quote">Le 20 juil. 2015 15:31, "Jozef Hartinger" <<a href="mailto:jharting@redhat.com">jharting@redhat.com</a>> a écrit :<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">As Martin explained, it is not the case that "remote EJBs are not CDI<br>
beans". Therefore, I am wondering why this behavior is a problem? AFAIK<br>
all the CDI implementations pass this test and therefore there may (at<br>
least in theory) be applications relying on this behavior.<br>
<br>
Jozef<br>
<br>
On 16.7.2015 15:25, Antoine Sabot-Durand wrote:<br>
> Hi guys,<br>
><br>
><br>
> Bill Shanon, just pointed me to this test in TCK:<br>
><br>
> <a href="https://github.com/cdi-spec/cdi-tck/blob/1.2/impl/src/main/java/org/jboss/cdi/tck/tests/event/observer/resolve/enterprise/ResolveEnterpriseEventObserverTest.java" rel="noreferrer" target="_blank">https://github.com/cdi-spec/cdi-tck/blob/1.2/impl/src/main/java/org/jboss/cdi/tck/tests/event/observer/resolve/enterprise/ResolveEnterpriseEventObserverTest.java</a><br>
><br>
> It tests the following assertion:<br>
><br>
> "If the bean is a session bean, the observer method must be either a<br>
> business method of the EJB or a static method of the bean class.”<br>
><br>
> The EJB containing the observers for the test is:<br>
><br>
> <a href="https://github.com/cdi-spec/cdi-tck/blob/1.2/impl/src/main/java/org/jboss/cdi/tck/tests/event/observer/resolve/enterprise/Spitz.java" rel="noreferrer" target="_blank">https://github.com/cdi-spec/cdi-tck/blob/1.2/impl/src/main/java/org/jboss/cdi/tck/tests/event/observer/resolve/enterprise/Spitz.java</a><br>
><br>
> The EJB contains one business method coming from a local interface, one<br>
> from a remote and one static method. the 3 are observers methods<br>
><br>
> The test expects that the remote business observer method should be called.<br>
><br>
> Here we have an inconsistency IMO. By doing this we are violating rules<br>
> the CDI spec regarding the fact that remote EJBs are not CDI beans. And<br>
> we are calling this remote business method passing the event payload by<br>
> reference and not by value which violates EJB specification regarding<br>
> remote EJB.<br>
><br>
> I suggest that we change the assertion to:<br>
><br>
> If the bean is a session bean, the observer method must be either a<br>
> local business method of the EJB or a static method of the bean class.<br>
><br>
> and the TCK accordingly. Any thought ?<br>
><br>
> Antoine<br>
><br>
><br>
> _______________________________________________<br>
> cdi-dev mailing list<br>
> <a href="mailto:cdi-dev@lists.jboss.org">cdi-dev@lists.jboss.org</a><br>
> <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
><br>
> Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (<a href="http://www.apache.org/licenses/LICENSE-2.0.html" rel="noreferrer" target="_blank">http://www.apache.org/licenses/LICENSE-2.0.html</a>). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.<br>
><br>
_______________________________________________<br>
cdi-dev mailing list<br>
<a href="mailto:cdi-dev@lists.jboss.org">cdi-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/cdi-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
<br>
Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (<a href="http://www.apache.org/licenses/LICENSE-2.0.html" rel="noreferrer" target="_blank">http://www.apache.org/licenses/LICENSE-2.0.html</a>). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.<br>
</blockquote></div>