<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p>In this specific case, that may be what's happening. I also am seeing it on Entity classes.</p>
<p><br>
</p>
<div id="Signature">
<div id="divtagdefaultwrapper" dir="ltr" style="font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255); font-family: Calibri, Arial, Helvetica, sans-serif, Helvetica, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">
<p></p>
<div style="color:rgb(33,33,33); font-size:15px; margin:0px">
<p style="margin:0in 0in 0.0001pt; font-size:11pt; font-family:Calibri"><span style="font-size:10.0pt; font-family:Arial; color:#3295B5">John D. Ament</span><span style="font-size:10.0pt; font-family:Arial; color:#003366"><br>
</span><span style="font-size:10.0pt; font-family:Arial; color:#808175">Cloud Software Architect</span><b><span style="font-size:10.0pt; font-family:Arial; color:#879637"></span></b></p>
<p style="margin:0in 0in 0.0001pt; font-size:11pt; font-family:Calibri"><span style="font-size:10.0pt; font-family:Arial; color:#808175">Sparta Systems<br>
</span><span style="font-size:10.0pt; font-family:Arial; color:#1E487C"><br>
</span><span style="font-size:10.0pt; font-family:Arial; color:#808175">p. 609.807.5466
</span><span style="font-size:10.0pt; font-family:Arial; color:#3295B5">|</span><span style="font-size:10.0pt; font-family:Arial; color:#2E74B5">
</span><span style="font-size:10.0pt; font-family:Arial; color:#808175">m.</span><span style="font-size:10.0pt; font-family:Arial; color:#2E74B5">
</span><span style="font-size:10.0pt; font-family:Arial; color:#808175">609.553.6130</span></p>
<p style="margin:0in 0in 0.0001pt; font-size:11pt; font-family:Calibri"><span style="color:rgb(5,99,193); text-decoration:underline"><span style="font-size:10.0pt; font-family:Arial; color:#7F7F7F"><a href="mailto:john.ament@spartasystems.com" id="LPNoLP"><span style="color:rgb(117,123,128)">john.ament@spartasystems.com</span></a> </span></span><span style="font-size:10.0pt; font-family:Arial; color:#3295B5">|</span><span style="font-size:10.0pt; font-family:Arial; color:#2E74B5">
</span><span style="color:rgb(127,127,127); text-decoration:underline; font-size:10pt; font-family:Arial"><a href="http://www.spartasystems.com" id="LPNoLP"><span style="color:rgb(117,123,128)">www.spartasystems.com</span></a><a href="http://www.spartasystems.com/" id="LPNoLP"></a></span><span style="font-size:10.0pt; font-family:Arial; color:#7F7F7F"></span></p>
<br>
</div>
<div style="color:rgb(33,33,33); font-size:15px; margin:0px"><br>
</div>
<p></p>
</div>
</div>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Matej Novotny <manovotn@redhat.com><br>
<b>Sent:</b> Monday, July 24, 2017 10:36 AM<br>
<b>To:</b> John Ament<br>
<b>Cc:</b> Tomas Remes; cdi-dev<br>
<b>Subject:</b> Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Hi John,<br>
<br>
I took a look at the Hammock test and I think you are sailing unspecified waters, so to say.<br>
You are right that Weld picks up DefaultListener as a bean even without BDA.<br>
<br>
In spec 2.0 section 26.2, there is this:<br>
"In Java EE, the container automatically discovers EJB session beans and **other Java EE component class supporting injection**, in bean archives like it does for managed bean as defined in Type and Bean discovery."<br>
<br>
So in EE, Listeners should be picked up with `annotated` even without having BDA.<br>
<br>
Now, hammock is using a combination of Weld SE + Servlet right?<br>
So the EE isn't your case, but then again spec doesn't talk about servlet at all.<br>
Therefore, I would say this is up to implementations to decide.<br>
<br>
Thoughts,opinions?<br>
<br>
Matej<br>
<br>
----- Original Message -----<br>
> From: "John Ament" <john.ament@spartasystems.com><br>
> To: "Tomas Remes" <tremes@redhat.com><br>
> Cc: "Matej Novotny" <manovotn@redhat.com>, "cdi-dev" <cdi-dev@lists.jboss.org><br>
> Sent: Monday, July 24, 2017 2:49:53 PM<br>
> Subject: Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode<br>
> <br>
> Sorry, should have been "DefaultListener"<br>
> <br>
> <br>
> John<br>
> <br>
> <br>
> ________________________________<br>
> From: Tomas Remes <tremes@redhat.com><br>
> Sent: Monday, July 24, 2017 7:59 AM<br>
> To: John Ament<br>
> Cc: Matej Novotny; cdi-dev<br>
> Subject: Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode<br>
> <br>
> <br>
> DefaultServlet appears to have @Dependent annotation.<br>
> <br>
> <a href="https://github.com/hammock-project/hammock/blob/hammock-1.5/web-tck/src/main/java/ws/ament/hammock/web/tck/DefaultServlet.java#L31">
https://github.com/hammock-project/hammock/blob/hammock-1.5/web-tck/src/main/java/ws/ament/hammock/web/tck/DefaultServlet.java#L31</a><br>
> <br>
> T.<br>
> <br>
> ----- Original Message -----<br>
> From: "John Ament" <john.ament@spartasystems.com><br>
> To: "Matej Novotny" <manovotn@redhat.com><br>
> Cc: "cdi-dev" <cdi-dev@lists.jboss.org><br>
> Sent: Monday, July 24, 2017 1:35:16 PM<br>
> Subject: Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode<br>
> <br>
> <br>
> <br>
> The issue has to do with ProcessAnnotatedType firing in a bean archive of<br>
> discovery mode "annotated." I think we all agree that if the discovery mode<br>
> is all with a trim flag, it works correctly.<br>
> <br>
> <br>
> <br>
> <br>
> It appears that if you have an Arquillian test in Weld, where a class has no<br>
> bean defining annotations, it still has a PAT fired. If you look at<br>
> <a href="https://github.com/hammock-project/hammock/blob/hammock-1.5/web-tck/src/main/java/ws/ament/hammock/web/tck/ServletTest.java">
https://github.com/hammock-project/hammock/blob/hammock-1.5/web-tck/src/main/java/ws/ament/hammock/web/tck/ServletTest.java</a><br>
> you'll see it includes a DefaultServlet class, which has no bean defining<br>
> annotations. In Weld, this generates a PAT, but in OWB it does not.<br>
> <br>
> <br>
> <br>
> <br>
> John<br>
> <br>
> <br>
> <br>
> <br>
> <br>
> <br>
> From: Matej Novotny <manovotn@redhat.com><br>
> Sent: Monday, July 24, 2017 6:38 AM<br>
> To: John Ament<br>
> Cc: Antoine Sabot-Durand; Martin Kouba; cdi-dev<br>
> Subject: Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode<br>
> What case exactly do you think Weld does incorrectly?<br>
> <br>
> If you have beans.xml `all` with `<trimm/>`, you should be firing PAT for<br>
> all. That's what me and Martin said.<br>
> <br>
> Matej<br>
> <br>
> ----- Original Message -----<br>
> > From: "John Ament" <john.ament@spartasystems.com><br>
> > To: "Antoine Sabot-Durand" <asd@redhat.com>, "Martin Kouba"<br>
> > <mkouba@redhat.com><br>
> > Cc: "cdi-dev" <cdi-dev@lists.jboss.org><br>
> > Sent: Monday, July 24, 2017 12:09:16 PM<br>
> > Subject: Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode<br>
> ><br>
> ><br>
> ><br>
> > Then that means Weld is doing it wrong (firing PAT for all classes).<br>
> ><br>
> ><br>
> ><br>
> ><br>
> > John<br>
> ><br>
> > From: Antoine Sabot-Durand <asd@redhat.com><br>
> > Sent: Monday, July 24, 2017 4:31 AM<br>
> > To: Martin Kouba<br>
> > Cc: John Ament; cdi-dev<br>
> > Subject: Re: [cdi-dev] ProcessAnnotatedType & Bean Discovery mode<br>
> > Yes it’s the correct interpretation.<br>
> > Confusion may come from the fact that we first considered to specify this<br>
> > feature the other way around (discover all types on implicites bean archive<br>
> > but only keep classes with BDA for bean discovery). We switched to this<br>
> > approach because we thought it was better for backward compatibility.<br>
> ><br>
> > Antoine<br>
> ><br>
> > > Le 24 juil. 2017 à 08:27, Martin Kouba <mkouba@redhat.com> a écrit :<br>
> > ><br>
> > > Hi John,<br>
> > ><br>
> > > I think OWB interpretation is correct.<br>
> > ><br>
> > > In an implicit bean archive (implicit = bean archive with<br>
> > > bean-discovery-mode=annotated) PAT is only fired for discovered types<br>
> > > and the container only discovers each class with a bean defining<br>
> > > annotation.<br>
> > ><br>
> > > For trimmed bean archive - only explicit bean archives may be marked as<br>
> > > 'trimmed'. PAT is fired there but if it does not have a bean defining<br>
> > > annotation it's removed from the set of discovered types, i.e. no bean<br>
> > > is created.<br>
> > ><br>
> > > Martin<br>
> > ><br>
> > > Dne 23.7.2017 v 14:32 John Ament napsal(a):<br>
> > >> Hi,<br>
> > >><br>
> > >><br>
> > >> An issue popped up in OWB wanted to get the EG's perspective on this.<br>
> > >><br>
> > >><br>
> > >> I have a given bean archive, with a class in it (no bean defining<br>
> > >> annotations). My interpretation of [1] and [2] is that this class is<br>
> > >> fired as a ProcessAnnotatedType, regardless of what is in beans.xml. In<br>
> > >> OWB, if I do bean-discovery-mode=annotated it does not get fired, but if<br>
> > >> it is all, even with a <trim/> a PAT gets fired. However it seems that<br>
> > >> it should get fired before anything is even considered about the class,<br>
> > >> and whether it has annotations.<br>
> > >><br>
> > >><br>
> > >> [1]:<br>
> > >> <a href="https://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#process_annotated_type">
https://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#process_annotated_type</a><br>
> ><br>
> ><br>
> > Contexts and Dependency Injection for the Java EE platform<br>
> > docs.jboss.org<br>
> > A well-defined lifecycle for stateful objects bound to lifecycle contexts,<br>
> > where the set of contexts is extensible. A sophisticated, typesafe<br>
> > dependency injection ...<br>
> ><br>
> ><br>
> > >><br>
> > >> [2]:<br>
> > >> <a href="https://docs.jboss.org/cdi/spec/2.0/cdi-spec.html#process_annotated_type">
https://docs.jboss.org/cdi/spec/2.0/cdi-spec.html#process_annotated_type</a><br>
> > >><br>
> > >><br>
> > >> John<br>
> > >><br>
> > >><br>
> > >> ------------------------------------------------------------------------<br>
> > >> NOTICE: This e-mail message and any attachments may contain<br>
> > >> confidential, proprietary, and/or privileged information which should be<br>
> > >> treated accordingly. If you are not the intended recipient, please<br>
> > >> notify the sender immediately by return e-mail, delete this message, and<br>
> > >> destroy all physical and electronic copies. Thank you.<br>
> > >><br>
> > >><br>
> > >> _______________________________________________<br>
> > >> cdi-dev mailing list<br>
> > >> cdi-dev@lists.jboss.org<br>
> > >> <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
> ><br>
> ><br>
> > cdi-dev Info Page - lists.jboss.org Mailing Lists<br>
> > lists.jboss.org<br>
> > List to discuss the development of CDI (the specification) To see the<br>
> > collection of prior postings to the list, visit the cdi-dev Archives.<br>
> ><br>
> ><br>
> > >><br>
> > >> Note that for all code provided on this list, the provider licenses the<br>
> > >> code under the Apache License, Version 2 (<br>
> > >> <a href="http://www.apache.org/licenses/LICENSE-2.0.html">http://www.apache.org/licenses/LICENSE-2.0.html</a> ). For all other ideas<br>
> > >> provided on this list, the provider waives all patent and other<br>
> > >> intellectual property rights inherent in such information.<br>
> > >><br>
> > ><br>
> > > --<br>
> > > Martin Kouba<br>
> > > Senior Software Engineer<br>
> > > Red Hat, Czech Republic<br>
> > > _______________________________________________<br>
> > > cdi-dev mailing list<br>
> > > cdi-dev@lists.jboss.org<br>
> > > <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
> > ><br>
> > > Note that for all code provided on this list, the provider licenses the<br>
> > > code under the Apache License, Version 2 (<br>
> > > <a href="http://www.apache.org/licenses/LICENSE-2.0.html">http://www.apache.org/licenses/LICENSE-2.0.html</a> ). For all other ideas<br>
> > > provided on this list, the provider waives all patent and other<br>
> > > intellectual property rights inherent in such information.<br>
> ><br>
> ><br>
> > NOTICE: This e-mail message and any attachments may contain confidential,<br>
> > proprietary, and/or privileged information which should be treated<br>
> > accordingly. If you are not the intended recipient, please notify the<br>
> > sender<br>
> > immediately by return e-mail, delete this message, and destroy all physical<br>
> > and electronic copies. Thank you.<br>
> ><br>
> > _______________________________________________<br>
> > cdi-dev mailing list<br>
> > cdi-dev@lists.jboss.org<br>
> > <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
> ><br>
> > Note that for all code provided on this list, the provider licenses the<br>
> > code<br>
> > under the Apache License, Version 2<br>
> > ( <a href="http://www.apache.org/licenses/LICENSE-2.0.html">http://www.apache.org/licenses/LICENSE-2.0.html</a> ). For all other ideas<br>
> > provided on this list, the provider waives all patent and other<br>
> > intellectual<br>
> > property rights inherent in such information.<br>
> <br>
> NOTICE: This e-mail message and any attachments may contain confidential,<br>
> proprietary, and/or privileged information which should be treated<br>
> accordingly. If you are not the intended recipient, please notify the sender<br>
> immediately by return e-mail, delete this message, and destroy all physical<br>
> and electronic copies. Thank you.<br>
> <br>
> _______________________________________________<br>
> cdi-dev mailing list<br>
> cdi-dev@lists.jboss.org<br>
> <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev">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<br>
> under the Apache License, Version 2<br>
> (<a href="http://www.apache.org/licenses/LICENSE-2.0.html">http://www.apache.org/licenses/LICENSE-2.0.html</a>). For all other ideas<br>
> provided on this list, the provider waives all patent and other intellectual<br>
> property rights inherent in such information.<br>
> <br>
> --<br>
> Tomas Remes<br>
> <br>
> <br>
> ________________________________<br>
> NOTICE: This e-mail message and any attachments may contain confidential,<br>
> proprietary, and/or privileged information which should be treated<br>
> accordingly. If you are not the intended recipient, please notify the sender<br>
> immediately by return e-mail, delete this message, and destroy all physical<br>
> and electronic copies. Thank you.<br>
> <br>
</div>
</span></font></div>
</div>
<hr>
NOTICE: This e-mail message and any attachments may contain confidential, proprietary, and/or privileged information which should be treated accordingly. If you are not the intended recipient, please notify the sender immediately by return e-mail, delete this
message, and destroy all physical and electronic copies. Thank you.
</body>
</html>