<div dir="ltr">I think we should just be importing the EL 3.0 API into our github repo and using the resulting org.jboss.spec artifacts for the API. <div><br></div><div>Looking at the repo it looks like this has already been imported, is there any reason we can't just use this? <br>
<div><div><br></div><div>Stuart</div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 29, 2013 at 7:25 PM, Shelly McGowan <span dir="ltr"><<a href="mailto:smcgowan@redhat.com" target="_blank">smcgowan@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On a related note to this discussion about EL 3.0 APIs:<br>
<a href="http://lists.jboss.org/pipermail/jboss-as7-dev/2013-July/008179.html" target="_blank">http://lists.jboss.org/pipermail/jboss-as7-dev/2013-July/008179.html</a><br>
<br>
See below a previous exchange on an attempt to remove EL 2.2 API dependency from WildFly.<br>
<br>
<br>
Shelly McGowan<br>
JBoss, by Red Hat<br>
<br>
<br>
<br>
<br>
> ----- Forwarded Message -----<br>
> From: "Stuart Douglas" <<a href="mailto:sdouglas@redhat.com">sdouglas@redhat.com</a>><br>
> To: "Shelly McGowan" <<a href="mailto:smcgowan@redhat.com">smcgowan@redhat.com</a>><br>
> Cc: "Remy Maucherat" <<a href="mailto:rmaucher@redhat.com">rmaucher@redhat.com</a>><br>
> Sent: Thursday, July 18, 2013 1:00:15 AM<br>
> Subject: Re: jboss-el-api_2.2_spec APIs<br>
><br>
> I created this class to address a fairly major EL performance issue (that factory finder is invoked on every EL invocation, and it is both slow and acquires locks, which cause lots of contention).<br>
><br>
> We may need to do a similar performance fix in the org.glassfish EL module, but I am not really sure. Assuming the problem still exists and we remove this fix our JSF performance will drop considerably.<br>
><br>
> Stuart<br>
><br>
> ----- Original Message -----<br>
>> From: "Shelly McGowan" <<a href="mailto:smcgowan@redhat.com">smcgowan@redhat.com</a>><br>
>> To: "Remy Maucherat" <<a href="mailto:rmaucher@redhat.com">rmaucher@redhat.com</a>><br>
>> Cc: "Stuart Douglas" <<a href="mailto:sdouglas@redhat.com">sdouglas@redhat.com</a>><br>
>> Sent: Thursday, 18 July, 2013 9:45:01 AM<br>
>> Subject: jboss-el-api_2.2_spec APIs<br>
>><br>
>><br>
>><br>
>> Remy,<br>
>><br>
>> Currently in WildFly we are using the org.glassfish:javax.el dependency for<br>
>> EL 3.0 APIs and implementation.<br>
>> I'm assuming the plan is to keep this in for at least WildFly 8.0.0.Final<br>
>> releases (and EE 7 certification) due to Beta1 only a few weeks away.<br>
>><br>
>> I was working on a pull request to remove the EL 2.2 APIs from WildFly and<br>
>> found out I was unable to do that due to a dependency<br>
>> from both the web module and the undertow module on:<br>
>> org.jboss.el.cache.FactoryFinderCache specifically in:<br>
>> undertow/src/main/java/org/wildfly/extension/undertow/deployment/ELExpressionFactoryProcessor.java<br>
>> web/src/main/java/org/jboss/as/web/deployment/ELExpressionFactoryProcessor.java<br>
>><br>
>> This class is included in the jboss-el-api Specs project API .jar but<br>
>> obviously not part of org.glassfish:javax.el.<br>
>><br>
>> How do you want to handle this moving forward? For tests purposes, I created<br>
>> an org.wildfly.el module to include only this class as you can see on this<br>
>> branch:<br>
>> <a href="https://github.com/smcgowan/wildfly/commit/3c511ea2e74a22215f02d69ff0d07d32bafb798d" target="_blank">https://github.com/smcgowan/wildfly/commit/3c511ea2e74a22215f02d69ff0d07d32bafb798d</a><br>
>><br>
>> [NOTE: there are a couple of unrelated changes in that branch]<br>
>><br>
>><br>
>> This work was prompted by some signature test failures I was seeing in TCK.<br>
>> The EL 2.2 APIs being in the distribution were unrelated.<br>
>><br>
>><br>
>> Shelly McGowan<br>
>> JBoss, by Red Hat<br>
<br>
_______________________________________________<br>
jboss-as7-dev mailing list<br>
<a href="mailto:jboss-as7-dev@lists.jboss.org">jboss-as7-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/jboss-as7-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/jboss-as7-dev</a><br>
</blockquote></div><br></div>