<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>This naming processor, which sets up a DU attachment for a set of jndi binding service dependencies (to be added later on the component start service so JNDI bindings are up when the component needs), is currently bound to INSTALL phase, and this is limiting the usage of such set. For instance in <a href="https://issues.jboss.org/browse/AS7-5443">https://issues.jboss.org/browse/AS7-5443</a> the naming processors who set up app and module context, which are bound to POST_MODULE phase, need to setup java:app/AppName and java:module/ModuleName, and these need to be also up when the component needs (e.g. @PreDestroy).</div><div><br></div><div>I see two ways of minimise such limitations:</div><div><br></div><div>a) This attachment is a set, which needs concrete setup, we could change it to a list, and take advantage of Attachable#addToAttachmentList(...), which inits the attachment list on demand. AFAIK the change from set to list would not be an issue, cause duplicated entries in this list would be MSC filtered/ignored (anyway we could always introduce Attachable#addToAttachmentSet(...)). In the case this has no side effects, we could even move the remaining logic of the processor to JndiNamingDependencyProcessor and remove it.</div><div><br></div><div>b) Move the Processor to an earlier phase, to ensure the attachment set is already setup.</div><div><br></div><div>Opinions?</div><br><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">-- E</div></span></span>
</div>
<br></body></html>