Re: ognl 2.7.3 performance

2009-11-06 Thread Dale Newfield
Chris Pratt wrote: In the struts.xml file you can use ${} to run an OGNL expression and access things from the Action (actually the value stack, but we're trying to keep it simple here) JSYK, %{} now works as expected in struts.xml. -Dale

Re: ognl 2.7.3 performance

2009-11-06 Thread Chris Pratt
Good to know, thanks. When was that changed? I use 2.1.6 in my personal project, but we're still stuck on 2.0.14 at work. (*Chris*) On Fri, Nov 6, 2009 at 11:33 PM, Dale Newfield d...@newfield.org wrote: Chris Pratt wrote: In the struts.xml file you can use ${} to run an OGNL expression

Re: ognl 2.7.3 performance

2009-11-06 Thread Musachy Barroso
in 2.1.6 I think. On Fri, Nov 6, 2009 at 11:47 PM, Chris Pratt thechrispr...@gmail.com wrote: Good to know, thanks.  When was that changed?  I use 2.1.6 in my personal project, but we're still stuck on 2.0.14 at work.  (*Chris*) On Fri, Nov 6, 2009 at 11:33 PM, Dale Newfield

Re: ognl 2.7.3 performance

2009-11-05 Thread Musachy Barroso
On Sat, Jul 18, 2009 at 3:06 PM, Musachy Barroso musa...@gmail.com wrote: Due to this fix: http://jira.opensymphony.com/browse/OGNL-141 Ognl 2.7.3 performance is a lot better than 2.6.11, on my rough test, the avg time spent in OgnlRuntime.invokeMethod(...) dropped from 630,726 ms (2.6.11

Re: ognl 2.7.3 performance

2009-11-05 Thread Chris Pratt
(more like copied, merging code from xwork and OGNL) a ReflectionProvider which does not need OGNL. //all hail our new UEL overlord musachy On Sat, Jul 18, 2009 at 3:06 PM, Musachy Barroso musa...@gmail.com wrote: Due to this fix: http://jira.opensymphony.com/browse/OGNL-141 Ognl 2.7.3

Re: ognl 2.7.3 performance

2009-11-04 Thread Musachy Barroso
ah..I didnt make it up after all: http://blogs.sun.com/kchung/entry/jsr_245_mr_part_i On Tue, Nov 3, 2009 at 4:04 PM, Musachy Barroso musa...@gmail.com wrote: yeah I have to look into that, it seems like the last jar in their maven repo doesn't have the latest api. The JUEL demo now works with

Re: ognl 2.7.3 performance

2009-11-03 Thread Antonio Petrelli
2009/11/3 Musachy Barroso musa...@gmail.com: I think I saw it in someone's blog but now I can't find it..did I make this up? Anyway, I am out of pet projects, so I might just play with a UEL plugin :) Is it really necessary? Isn't it possible to let EL be used by the container? In Tiles, in

Re: ognl 2.7.3 performance

2009-11-03 Thread Musachy Barroso
We also have FreeMarker , Velocity and we have a lot of expression evaluations from Struts code itself. musachy On Tue, Nov 3, 2009 at 12:29 AM, Antonio Petrelli antonio.petre...@gmail.com wrote: 2009/11/3 Musachy Barroso musa...@gmail.com: I think I saw it in someone's blog but now I can't

Re: ognl 2.7.3 performance

2009-11-03 Thread Antonio Petrelli
2009/11/3 Musachy Barroso musa...@gmail.com: We also have FreeMarker , Velocity and we have a lot of expression evaluations from Struts code itself. And in this case you're right, EL at Struts-side is obligatory. But exactly, is a bad idea to use the capability of the container to resolve EL

Re: ognl 2.7.3 performance

2009-11-03 Thread Brian Pontarelli
I still think that the simplest approach is still to do nothing except for EL and let the view technology do it all (JSP, FTL, VM, etc.). The only time you need anything remotely similar to EL is for getting and setting values out of beans. This is generally not EL handling, but basic

Re: ognl 2.7.3 performance

2009-11-03 Thread Musachy Barroso
That would be ok except for one thing: the value stack. To support the value stack in those view technologies is the problem. I have tried so many things, and failed in all of them that it is lame. I will finally merge my parameters-binder branch in xwork into trunk, and see if I can get some

Re: ognl 2.7.3 performance

2009-11-03 Thread Brian Pontarelli
It's been a while, but that is really more of an current action stack (I call it the ActionInvocation stack). I recall that I was able to get most of the functionality I needed into JCatapult while still using the FTL/JSP expression languages. -bp On Nov 3, 2009, at 11:10 AM, Musachy

Re: ognl 2.7.3 performance

2009-11-03 Thread Musachy Barroso
The parameters binder branch is now merged into xwork trunk (manual merge thanks to svn being a PITA)(for those unaware, this contains some experimental code to set parameters in the actions without using OGNL directly, it is faster, and more secure) . Now we can start playing with other ELs

Re: ognl 2.7.3 performance

2009-11-03 Thread Antonio Petrelli
2009/11/3 Musachy Barroso musa...@gmail.com: Well yes, that's by default, but with the new EL api you can plugin a new EL resolver like:  JspApplicationContext jspApplicationContext = JspFactory.getDefaultFactory().getJspApplicationContext(servletContext);  

Re: ognl 2.7.3 performance

2009-11-02 Thread Chris Pratt
Oops, I must not have looked close enough, I'm using OGNL 2.7.3 with javassist-3.7.0.jar. (*Chris*) On Mon, Nov 2, 2009 at 3:39 PM, Musachy Barroso musa...@gmail.com wrote: Chris, I wanted to double check on this, are you using Ognl 2.7.3 without Javassist? I get class loading errors without

Re: ognl 2.7.3 performance

2009-11-02 Thread Musachy Barroso
It would be sweet if javassist was optional in OGNL, specially considering that we do not use it. I will resist the temptation to vent my frustrations with OGNL here :) (and to fork it as well) musachy On Mon, Nov 2, 2009 at 4:42 PM, Chris Pratt thechrispr...@gmail.com wrote: Oops, I must not

Re: ognl 2.7.3 performance

2009-11-02 Thread Chris Pratt
It would be sweeter if OGNL was optional in struts, but that topic has been beaten to death. =8^( (*Chris*) On Mon, Nov 2, 2009 at 5:13 PM, Musachy Barroso musa...@gmail.com wrote: It would be sweet if javassist was optional in OGNL, specially considering that we do not use it. I will resist

Re: ognl 2.7.3 performance

2009-11-02 Thread Musachy Barroso
Actually is not as far off as it sounds. Using the parameters-binder branch and mvel branch in xwork you get a half working showcase. I just need to get myself to do it :) musachy On Mon, Nov 2, 2009 at 6:14 PM, Chris Pratt thechrispr...@gmail.com wrote: It would be sweeter if OGNL was optional

Re: ognl 2.7.3 performance

2009-11-02 Thread Chris Pratt
I know it's not as sexy, but at this point I think I'd prefer plain old JSTL EL to work globally. It's much easier to teach my junior programmers a single EL than have to explain where each is appropriate. But maybe that's just me. (*Chris*) On Mon, Nov 2, 2009 at 7:06 PM, Musachy Barroso

Re: ognl 2.7.3 performance

2009-11-02 Thread Musachy Barroso
Thy are finally adding static method calls finally. I don't use any advanced features from OGNL either, UEL would do just fine for me. On Mon, Nov 2, 2009 at 7:20 PM, Chris Pratt thechrispr...@gmail.com wrote: I know it's not as sexy, but at this point I think I'd prefer plain old JSTL EL to

Re: ognl 2.7.3 performance

2009-11-02 Thread Chris Pratt
They've had static method calls for a long time, you just had to define them as Functions in the TLD. Have they added something else new? (*Chris*) On Mon, Nov 2, 2009 at 8:36 PM, Musachy Barroso musa...@gmail.com wrote: Thy are finally adding static method calls finally. I don't use any

Re: ognl 2.7.3 performance

2009-11-02 Thread Musachy Barroso
I think I saw it in someone's blog but now I can't find it..did I make this up? Anyway, I am out of pet projects, so I might just play with a UEL plugin :) musachy On Mon, Nov 2, 2009 at 8:42 PM, Chris Pratt thechrispr...@gmail.com wrote: They've had static method calls for a long time, you

Re: ognl 2.7.3 performance

2009-07-29 Thread Rainer Hermanns
:06 PM, Musachy Barrosomusa...@gmail.com wrote: Due to this fix: http://jira.opensymphony.com/browse/OGNL-141 Ognl 2.7.3 performance is a lot better than 2.6.11, on my rough test, the avg time spent in OgnlRuntime.invokeMethod(...) dropped from 630,726 ms (2.6.11) to 117,108 (2.7.3) ms, (100

Re: ognl 2.7.3 performance

2009-07-29 Thread Lukasz Lenart
2009/7/29 Rainer Hermanns herma...@aixcept.de: thanks! I'll wait for the TextProvider problem to be fixed and then fire up the release process. I start a new gig next week, so this will most likely happen until/over the weekend. I solved these problems and was able to confirm that with test,

Re: ognl 2.7.3 performance

2009-07-27 Thread Musachy Barroso
is ok, we can release xwork and then do the 2.1.8 release. musachy On Sat, Jul 18, 2009 at 4:06 PM, Musachy Barrosomusa...@gmail.com wrote: Due to this fix: http://jira.opensymphony.com/browse/OGNL-141 Ognl 2.7.3 performance is a lot better than 2.6.11, on my rough test, the avg time spent

Re: ognl 2.7.3 performance

2009-07-23 Thread Musachy Barroso
: http://jira.opensymphony.com/browse/OGNL-141 Ognl 2.7.3 performance is a lot better than 2.6.11, on my rough test, the avg time spent in OgnlRuntime.invokeMethod(...) dropped from 630,726 ms (2.6.11) to 117,108 (2.7.3) ms, (100 threads, 2s ramp-up period, 100 iterations) . If we are not going

Re: ognl 2.7.3 performance

2009-07-20 Thread Rainer Hermanns
+1, both make sense to wait for, cause the performance improvement will satisfy lots of our users... That's the reason, why xwork is not yet released :) cheers, Rainer Sounds good. It gives rainer some more time to get xwork released. On 7/18/09, Dale Newfield d...@newfield.org wrote:

Re: ognl 2.7.3 performance

2009-07-20 Thread Musachy Barroso
hum..there is one minor problem, ognl 2.7.3 depends on jboss:javassist. musachy On Sun, Jul 19, 2009 at 11:32 PM, Rainer Hermannsherma...@aixcept.de wrote: +1, both make sense to wait for, cause the performance improvement will satisfy lots of our users... That's the reason, why xwork is not

Re: ognl 2.7.3 performance

2009-07-20 Thread Chris Pratt
That may be only for the compilation. I'm using it without those libraries and haven't had any problems. (*Chris*) On Mon, Jul 20, 2009 at 9:30 AM, Musachy Barroso musa...@gmail.com wrote: hum..there is one minor problem, ognl 2.7.3 depends on jboss:javassist. musachy On Sun, Jul 19,

Re: ognl 2.7.3 performance

2009-07-20 Thread Musachy Barroso
that's good to hear, I haven't tried it yet without that dep. musachy On Mon, Jul 20, 2009 at 9:46 AM, Chris Prattthechrispr...@gmail.com wrote: That may be only for the compilation.  I'm using it without those libraries and haven't had any problems.  (*Chris*) On Mon, Jul 20, 2009 at 9:30

Re: ognl 2.7.3 performance

2009-07-20 Thread Musachy Barroso
I need some maven black magic support, running mvn dependency:tree, from apps/blank outputs: [INFO] org.apache.struts:struts2-blank:war:2.1.8-SNAPSHOT [INFO] +- javax.servlet:servlet-api:jar:2.4:provided [INFO] +- javax.servlet:jsp-api:jar:2.0:provided [INFO] +-

ognl 2.7.3 performance

2009-07-18 Thread Musachy Barroso
Due to this fix: http://jira.opensymphony.com/browse/OGNL-141 Ognl 2.7.3 performance is a lot better than 2.6.11, on my rough test, the avg time spent in OgnlRuntime.invokeMethod(...) dropped from 630,726 ms (2.6.11) to 117,108 (2.7.3) ms, (100 threads, 2s ramp-up period, 100 iterations) . If we

Re: ognl 2.7.3 performance

2009-07-18 Thread Dave Newton
Dale Newfield wrote: Musachy Barroso musa...@gmail.com wrote: With the bytecode stuff out the way I am inclined to just upgrade to 2.7.3 at once, and upgrade freemarker also. +1 Also +1, and I'll have a bit of time this week. Dave

Re: ognl 2.7.3 performance

2009-07-18 Thread Dale Newfield
Musachy Barroso musa...@gmail.com wrote: With the bytecode stuff out the way I am inclined to just upgrade to 2.7.3 at once, and upgrade freemarker also. +1 -Dale - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org

Re: ognl 2.7.3 performance

2009-07-18 Thread Musachy Barroso
committed. here are the tickets: http://jira.opensymphony.com/browse/XW-710 https://issues.apache.org/struts/browse/WW-3198 Ognl 2.7.3 has a groupid of ognl instead of opensymphony. As a side note, I just noticed that trying to get a value from the ValueStack, whose key(or an intermediate

Re: ognl 2.7.3 performance

2009-07-18 Thread Wes Wannemacher
Sounds good. It gives rainer some more time to get xwork released. On 7/18/09, Dale Newfield d...@newfield.org wrote: Musachy Barroso musa...@gmail.com wrote: With the bytecode stuff out the way I am inclined to just upgrade to 2.7.3 at once, and upgrade freemarker also. +1 -Dale

Re: ognl 2.7.3 performance

2009-07-18 Thread Musachy Barroso
On Sat, Jul 18, 2009 at 5:43 PM, Musachy Barrosomusa...@gmail.com wrote: As a side note, I just noticed that trying to get a value from the ValueStack, whose key(or an intermediate object in the expression) does not exist is an expensive operation, because a new OgnlException is thrown, even

Re: ognl 2.7.3 performance

2009-07-18 Thread Chris Pratt
I've been using ognl 2.7.2 for quite a while with no problems. But when I saw you guys talking about 2.7.3, I decided to update my library and give it a try, but http://www.ognl.org seems to be gone and the download page on http://www.opensymphony.com/ognl seems to be down as well. Is there

Re: ognl 2.7.3 performance

2009-07-18 Thread Dave Newton
Chris Pratt wrote: I've been using ognl 2.7.2 for quite a while with no problems. But when I saw you guys talking about 2.7.3, I decided to update my library and give it a try, but http://www.ognl.org seems to be gone and the download page on http://www.opensymphony.com/ognl seems to be down as

Re: ognl 2.7.3 performance

2009-07-18 Thread Musachy Barroso
yup, you have to get it from maven repo, http://www.opensymphony.com/ognl/ works for me, but the download page is empty anyway. musachy On Sat, Jul 18, 2009 at 8:35 PM, Dave Newtonnewton.d...@yahoo.com wrote: Chris Pratt wrote: I've been using ognl 2.7.2 for quite a while with no problems.