<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>Then that means Weld is doing it wrong (firing PAT for&nbsp;all classes).</p>
<p><br>
</p>
<p>John</p>
<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> Antoine Sabot-Durand &lt;asd@redhat.com&gt;<br>
<b>Sent:</b> Monday, July 24, 2017 4:31 AM<br>
<b>To:</b> Martin Kouba<br>
<b>Cc:</b> John Ament; cdi-dev<br>
<b>Subject:</b> Re: [cdi-dev] ProcessAnnotatedType &amp; Bean Discovery mode</font>
<div>&nbsp;</div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Yes it’s the correct interpretation.<br>
Confusion may come from the fact that we first considered to specify this feature the other way around (discover all types on implicites bean archive but only keep classes with BDA for bean discovery). We switched to this approach because we thought it was
 better for backward compatibility.<br>
<br>
Antoine <br>
<br>
&gt; Le 24 juil. 2017 à 08:27, Martin Kouba &lt;mkouba@redhat.com&gt; a écrit :<br>
&gt; <br>
&gt; Hi John,<br>
&gt; <br>
&gt; I think OWB interpretation is correct.<br>
&gt; <br>
&gt; In an implicit bean archive (implicit = bean archive with <br>
&gt; bean-discovery-mode=annotated) PAT is only fired for discovered types <br>
&gt; and the container only discovers each class with a bean defining annotation.<br>
&gt; <br>
&gt; For trimmed bean archive - only explicit bean archives may be marked as <br>
&gt; 'trimmed'. PAT is fired there but if it does not have a bean defining <br>
&gt; annotation it's removed from the set of discovered types, i.e. no bean <br>
&gt; is created.<br>
&gt; <br>
&gt; Martin<br>
&gt; <br>
&gt; Dne 23.7.2017 v 14:32 John Ament napsal(a):<br>
&gt;&gt; Hi,<br>
&gt;&gt; <br>
&gt;&gt; <br>
&gt;&gt; An issue popped up in OWB wanted to get the EG's perspective on this.<br>
&gt;&gt; <br>
&gt;&gt; <br>
&gt;&gt; I have a given bean archive, with a class in it (no bean defining <br>
&gt;&gt; annotations).&nbsp; My interpretation of [1] and [2] is that this class is <br>
&gt;&gt; fired as a ProcessAnnotatedType, regardless of what is in beans.xml.&nbsp; In <br>
&gt;&gt; OWB, if I do bean-discovery-mode=annotated it does not get fired, but if <br>
&gt;&gt; it is all, even with a &lt;trim/&gt; a PAT gets fired.&nbsp; However it seems that <br>
&gt;&gt; it should get fired before anything is even considered about the class, <br>
&gt;&gt; and whether it has annotations.<br>
&gt;&gt; <br>
&gt;&gt; <br>
&gt;&gt; [1]: <br>
&gt;&gt; <a href="https://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#process_annotated_type" id="LPlnk839889" previewremoved="true">
https://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#process_annotated_type</a>
<div id="LPBorder_GT_15008909373300.07048380549995614" style="margin-bottom: 20px; overflow: auto; width: 100%; text-indent: 0px;">
<table id="LPContainer_15008909373280.2700424014208329" role="presentation" cellspacing="0" style="width: 90%; background-color: rgb(255, 255, 255); position: relative; overflow: auto; padding-top: 20px; padding-bottom: 20px; margin-top: 20px; border-top: 1px dotted rgb(200, 200, 200); border-bottom: 1px dotted rgb(200, 200, 200);">
<tbody>
<tr valign="top" style="border-spacing: 0px;">
<td id="TextCell_15008909373290.599612219758042" colspan="2" style="vertical-align: top; position: relative; padding: 0px; display: table-cell;">
<div id="LPRemovePreviewContainer_15008909373290.5763296182695887"></div>
<div id="LPTitle_15008909373290.3309200533891241" style="top: 0px; color: rgb(255, 140, 0); font-weight: normal; font-size: 21px; font-family: wf_segoe-ui_light, &quot;Segoe UI Light&quot;, &quot;Segoe WP Light&quot;, &quot;Segoe UI&quot;, &quot;Segoe WP&quot;, Tahoma, Arial, sans-serif; line-height: 21px;">
<a id="LPUrlAnchor_15008909373290.5089962185306551" href="https://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#process_annotated_type" target="_blank" style="text-decoration: none;">Contexts and Dependency Injection for the Java EE platform</a></div>
<div id="LPMetadata_15008909373290.6448071680835865" style="margin: 10px 0px 16px; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, &quot;Segoe UI&quot;, &quot;Segoe WP&quot;, Tahoma, Arial, sans-serif; font-size: 14px; line-height: 14px;">
docs.jboss.org</div>
<div id="LPDescription_15008909373300.7253422092069006" style="display: block; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, &quot;Segoe UI&quot;, &quot;Segoe WP&quot;, Tahoma, Arial, sans-serif; font-size: 14px; line-height: 20px; max-height: 100px; overflow: hidden;">
A well-defined lifecycle for stateful objects bound to lifecycle contexts, where the set of contexts is extensible. A sophisticated, typesafe dependency injection ...</div>
</td>
</tr>
</tbody>
</table>
</div>
<br>
<br>
&gt;&gt; <br>
&gt;&gt; [2]: <br>
&gt;&gt; <a href="https://docs.jboss.org/cdi/spec/2.0/cdi-spec.html#process_annotated_type" id="LPlnk346050" previewremoved="true">
https://docs.jboss.org/cdi/spec/2.0/cdi-spec.html#process_annotated_type</a><br>
&gt;&gt; <br>
&gt;&gt; <br>
&gt;&gt; John<br>
&gt;&gt; <br>
&gt;&gt; <br>
&gt;&gt; ------------------------------------------------------------------------<br>
&gt;&gt; NOTICE: This e-mail message and any attachments may contain <br>
&gt;&gt; confidential, proprietary, and/or privileged information which should be <br>
&gt;&gt; treated accordingly. If you are not the intended recipient, please <br>
&gt;&gt; notify the sender immediately by return e-mail, delete this message, and <br>
&gt;&gt; destroy all physical and electronic copies. Thank you.<br>
&gt;&gt; <br>
&gt;&gt; <br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; cdi-dev mailing list<br>
&gt;&gt; cdi-dev@lists.jboss.org<br>
&gt;&gt; <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev" id="LPlnk620867" previewremoved="true">
https://lists.jboss.org/mailman/listinfo/cdi-dev</a>
<div id="LPBorder_GT_15008909372570.5315426243695558" style="margin-bottom: 20px; overflow: auto; width: 100%; text-indent: 0px;">
<table id="LPContainer_15008909372540.23325419930978364" role="presentation" cellspacing="0" style="width: 90%; background-color: rgb(255, 255, 255); position: relative; overflow: auto; padding-top: 20px; padding-bottom: 20px; margin-top: 20px; border-top: 1px dotted rgb(200, 200, 200); border-bottom: 1px dotted rgb(200, 200, 200);">
<tbody>
<tr valign="top" style="border-spacing: 0px;">
<td id="TextCell_15008909372550.675325386837708" colspan="2" style="vertical-align: top; position: relative; padding: 0px; display: table-cell;">
<div id="LPRemovePreviewContainer_15008909372560.9409446120280429"></div>
<div id="LPTitle_15008909372560.8748714159882021" style="top: 0px; color: rgb(255, 140, 0); font-weight: normal; font-size: 21px; font-family: wf_segoe-ui_light, &quot;Segoe UI Light&quot;, &quot;Segoe WP Light&quot;, &quot;Segoe UI&quot;, &quot;Segoe WP&quot;, Tahoma, Arial, sans-serif; line-height: 21px;">
<a id="LPUrlAnchor_15008909372560.37091964995591864" href="https://lists.jboss.org/mailman/listinfo/cdi-dev" target="_blank" style="text-decoration: none;">cdi-dev Info Page - lists.jboss.org Mailing Lists</a></div>
<div id="LPMetadata_15008909372560.12566627288356869" style="margin: 10px 0px 16px; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, &quot;Segoe UI&quot;, &quot;Segoe WP&quot;, Tahoma, Arial, sans-serif; font-size: 14px; line-height: 14px;">
lists.jboss.org</div>
<div id="LPDescription_15008909372570.08573632256035868" style="display: block; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, &quot;Segoe UI&quot;, &quot;Segoe WP&quot;, Tahoma, Arial, sans-serif; font-size: 14px; line-height: 20px; max-height: 100px; overflow: hidden;">
List to discuss the development of CDI (the specification) To see the collection of prior postings to the list, visit the cdi-dev Archives.</div>
</td>
</tr>
</tbody>
</table>
</div>
<br>
<br>
&gt;&gt; <br>
&gt;&gt; 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" id="LPlnk766522">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>
&gt;&gt; <br>
&gt; <br>
&gt; -- <br>
&gt; Martin Kouba<br>
&gt; Senior Software Engineer<br>
&gt; Red Hat, Czech Republic<br>
&gt; _______________________________________________<br>
&gt; cdi-dev mailing list<br>
&gt; cdi-dev@lists.jboss.org<br>
&gt; <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
&gt; <br>
&gt; 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">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>
</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>