ognl 2.7.3 performance

View: New views
20 Messages — Rating Filter:   Alert me  
< Prev | 1 - 2 - 3 | Next >

ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 are not going to use the new bycode
stuff, then the upgrade doesn't seem that risky. Should we upgrade, do
some testing, and release 2.1.8 (delay release a bit), or leave that
for 2.1.9? With the bytecode stuff out the way I am inclined to just
upgrade to 2.7.3 at once, and upgrade freemarker also.

musachy

--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by DNewfield :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Musachy Barroso <musachy@...> 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-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Wes Wannemacher :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Sounds good. It gives rainer some more time to get xwork released.

On 7/18/09, Dale Newfield <dale@...> wrote:

> Musachy Barroso <musachy@...> 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-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>


--
Wes Wannemacher
Author - Struts 2 In Practice
Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
http://www.manning.com/wannemacher

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by newton.dave :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Dale Newfield wrote:
> Musachy Barroso <musachy@...> 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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 object in the expression)
does not exist is an expensive operation, because a new OgnlException
is thrown, even when it doesn't make it out of the value stack it is a
(unnecessary) performance hit, and it happens very, very often, I am
trying to figure out  way to avoid it.

regards
musachy

On Sat, Jul 18, 2009 at 5:08 PM, Wes Wannemacher<wesw@...> wrote:

> Sounds good. It gives rainer some more time to get xwork released.
>
> On 7/18/09, Dale Newfield <dale@...> wrote:
>> Musachy Barroso <musachy@...> 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-unsubscribe@...
>> For additional commands, e-mail: dev-help@...
>>
>>
>
>
> --
> Wes Wannemacher
> Author - Struts 2 In Practice
> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
> http://www.manning.com/wannemacher
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Sat, Jul 18, 2009 at 5:43 PM, Musachy Barroso<musachy@...> 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 when it doesn't make it out of the value stack it is a
> (unnecessary) performance hit, and it happens very, very often, I am
> trying to figure out  way to avoid it.

meh...not fixable, I would have to do a change on OGNL code.

musachy
--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Chris Pratt :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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
somewhere else I should be looking for the updated libraries?
  (*Chris*)

On Sat, Jul 18, 2009 at 6:02 PM, Musachy Barroso <musachy@...> wrote:

> On Sat, Jul 18, 2009 at 5:43 PM, Musachy Barroso<musachy@...> 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 when it doesn't make it out of the value stack it is a
> > (unnecessary) performance hit, and it happens very, very often, I am
> > trying to figure out  way to avoid it.
>
> meh...not fixable, I would have to do a change on OGNL code.
>
> musachy
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>

Re: ognl 2.7.3 performance

by newton.dave :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 well.  Is there
> somewhere else I should be looking for the updated libraries?

Can always get it from a Maven repo:

http://mirrors.ibiblio.org/pub/mirrors/maven2/ognl/ognl/

Dave

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 Newton<newton.dave@...> wrote:

> 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 well.  Is there
>> somewhere else I should be looking for the updated libraries?
>
> Can always get it from a Maven repo:
>
> http://mirrors.ibiblio.org/pub/mirrors/maven2/ognl/ognl/
>
> Dave
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Rainer Hermanns-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

+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 <dale@...> wrote:
>> Musachy Barroso <musachy@...> 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-unsubscribe@...
>> For additional commands, e-mail: dev-help@...
>>
>>
>
>
> --
> Wes Wannemacher
> Author - Struts 2 In Practice
> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
> http://www.manning.com/wannemacher
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>


--
Rainer Hermanns
aixcept
Willibrordstraße 82
52134 Herzogenrath - Germany
w: http://aixcept.de/
t: +49 - 2406 - 979 22 11
f: +49 - 2406 - 979 22 13
m: +49 - 170 - 343 29 12

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 Hermanns<hermanns@...> 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 yet released :)
>
> cheers,
> Rainer
>
>> Sounds good. It gives rainer some more time to get xwork released.
>>
>> On 7/18/09, Dale Newfield <dale@...> wrote:
>>> Musachy Barroso <musachy@...> 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-unsubscribe@...
>>> For additional commands, e-mail: dev-help@...
>>>
>>>
>>
>>
>> --
>> Wes Wannemacher
>> Author - Struts 2 In Practice
>> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
>> http://www.manning.com/wannemacher
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@...
>> For additional commands, e-mail: dev-help@...
>>
>>
>
>
> --
> Rainer Hermanns
> aixcept
> Willibrordstraße 82
> 52134 Herzogenrath - Germany
> w: http://aixcept.de/
> t: +49 - 2406 - 979 22 11
> f: +49 - 2406 - 979 22 13
> m: +49 - 170 - 343 29 12
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Chris Pratt :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 <musachy@...> wrote:

> 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 Hermanns<hermanns@...>
> 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 yet released :)
> >
> > cheers,
> > Rainer
> >
> >> Sounds good. It gives rainer some more time to get xwork released.
> >>
> >> On 7/18/09, Dale Newfield <dale@...> wrote:
> >>> Musachy Barroso <musachy@...> 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-unsubscribe@...
> >>> For additional commands, e-mail: dev-help@...
> >>>
> >>>
> >>
> >>
> >> --
> >> Wes Wannemacher
> >> Author - Struts 2 In Practice
> >> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
> >> http://www.manning.com/wannemacher
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@...
> >> For additional commands, e-mail: dev-help@...
> >>
> >>
> >
> >
> > --
> > Rainer Hermanns
> > aixcept
> > Willibrordstraße 82
> > 52134 Herzogenrath - Germany
> > w: http://aixcept.de/
> > t: +49 - 2406 - 979 22 11
> > f: +49 - 2406 - 979 22 13
> > m: +49 - 170 - 343 29 12
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@...
> > For additional commands, e-mail: dev-help@...
> >
> >
>
>
>
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>

Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 Pratt<thechrispratt@...> 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 AM, Musachy Barroso <musachy@...> wrote:
>
>> 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 Hermanns<hermanns@...>
>> 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 yet released :)
>> >
>> > cheers,
>> > Rainer
>> >
>> >> Sounds good. It gives rainer some more time to get xwork released.
>> >>
>> >> On 7/18/09, Dale Newfield <dale@...> wrote:
>> >>> Musachy Barroso <musachy@...> 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-unsubscribe@...
>> >>> For additional commands, e-mail: dev-help@...
>> >>>
>> >>>
>> >>
>> >>
>> >> --
>> >> Wes Wannemacher
>> >> Author - Struts 2 In Practice
>> >> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
>> >> http://www.manning.com/wannemacher
>> >>
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: dev-unsubscribe@...
>> >> For additional commands, e-mail: dev-help@...
>> >>
>> >>
>> >
>> >
>> > --
>> > Rainer Hermanns
>> > aixcept
>> > Willibrordstraße 82
>> > 52134 Herzogenrath - Germany
>> > w: http://aixcept.de/
>> > t: +49 - 2406 - 979 22 11
>> > f: +49 - 2406 - 979 22 13
>> > m: +49 - 170 - 343 29 12
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: dev-unsubscribe@...
>> > For additional commands, e-mail: dev-help@...
>> >
>> >
>>
>>
>>
>> --
>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@...
>> For additional commands, e-mail: dev-help@...
>>
>>
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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] +- org.apache.struts:struts2-junit-plugin:jar:2.1.8-SNAPSHOT:test
[INFO] |  \- org.springframework:spring-core:jar:2.5.6:test
[INFO] +- org.apache.struts:struts2-core:jar:2.1.8-SNAPSHOT:compile
[INFO] |  +- com.opensymphony:xwork-core:jar:2.1.5-SNAPSHOT:compile
[INFO] |  |  +- commons-lang:commons-lang:jar:2.4:compile
[INFO] |  |  +- opensymphony:ognl:jar:2.6.11:compile
[INFO] |  |  +- asm:asm:jar:3.1:compile
[INFO] |  |  \- asm:asm-commons:jar:3.1:compile
[INFO] |  |     \- asm:asm-tree:jar:3.1:compile
[INFO] |  +- org.freemarker:freemarker:jar:2.3.15:compile
[INFO] |  +- ognl:ognl:jar:2.7.3:compile
[INFO] |  +- commons-fileupload:commons-fileupload:jar:1.2.1:compile
[INFO] |  +- commons-io:commons-io:jar:1.3.2:compile
[INFO] |  \- com.sun:tools:jar:1.5.0:system
[INFO] \- org.springframework:spring-test:jar:2.5.6:test (scope not
updated to compile)
[INFO]    +- commons-logging:commons-logging:jar:1.1.1:test
[INFO]    \- junit:junit:jar:3.8.1:test

but the asm dependency is marked as "optional" in xwork, so it
shouldn't be resolved as a transitive dependency, anyone knows what's
up here?

musachy

On Mon, Jul 20, 2009 at 9:50 AM, Musachy Barroso<musachy@...> wrote:

> 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 Pratt<thechrispratt@...> 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 AM, Musachy Barroso <musachy@...> wrote:
>>
>>> 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 Hermanns<hermanns@...>
>>> 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 yet released :)
>>> >
>>> > cheers,
>>> > Rainer
>>> >
>>> >> Sounds good. It gives rainer some more time to get xwork released.
>>> >>
>>> >> On 7/18/09, Dale Newfield <dale@...> wrote:
>>> >>> Musachy Barroso <musachy@...> 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-unsubscribe@...
>>> >>> For additional commands, e-mail: dev-help@...
>>> >>>
>>> >>>
>>> >>
>>> >>
>>> >> --
>>> >> Wes Wannemacher
>>> >> Author - Struts 2 In Practice
>>> >> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
>>> >> http://www.manning.com/wannemacher
>>> >>
>>> >> ---------------------------------------------------------------------
>>> >> To unsubscribe, e-mail: dev-unsubscribe@...
>>> >> For additional commands, e-mail: dev-help@...
>>> >>
>>> >>
>>> >
>>> >
>>> > --
>>> > Rainer Hermanns
>>> > aixcept
>>> > Willibrordstraße 82
>>> > 52134 Herzogenrath - Germany
>>> > w: http://aixcept.de/
>>> > t: +49 - 2406 - 979 22 11
>>> > f: +49 - 2406 - 979 22 13
>>> > m: +49 - 170 - 343 29 12
>>> >
>>> > ---------------------------------------------------------------------
>>> > To unsubscribe, e-mail: dev-unsubscribe@...
>>> > For additional commands, e-mail: dev-help@...
>>> >
>>> >
>>>
>>>
>>>
>>> --
>>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@...
>>> For additional commands, e-mail: dev-help@...
>>>
>>>
>>
>
>
>
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

I am done with the changes in xwork. Whoever can give a hand testing
what is in trunk, this is a good time to jump in :). If everything 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 Barroso<musachy@...> 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 threads, 2s ramp-up
> period, 100 iterations) . If we are not going to use the new bycode
> stuff, then the upgrade doesn't seem that risky. Should we upgrade, do
> some testing, and release 2.1.8 (delay release a bit), or leave that
> for 2.1.9? With the bytecode stuff out the way I am inclined to just
> upgrade to 2.7.3 at once, and upgrade freemarker also.
>
> musachy
>
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

showcase seems fine. Rainer, whenever you can, get us an xwork release :)

musachy

On Thu, Jul 23, 2009 at 10:57 AM, Musachy Barroso<musachy@...> wrote:

> I am done with the changes in xwork. Whoever can give a hand testing
> what is in trunk, this is a good time to jump in :). If everything 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 Barroso<musachy@...> 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 threads, 2s ramp-up
>> period, 100 iterations) . If we are not going to use the new bycode
>> stuff, then the upgrade doesn't seem that risky. Should we upgrade, do
>> some testing, and release 2.1.8 (delay release a bit), or leave that
>> for 2.1.9? With the bytecode stuff out the way I am inclined to just
>> upgrade to 2.7.3 at once, and upgrade freemarker also.
>>
>> musachy
>>
>> --
>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>
>
>
>
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd
>



--
"Hey you! Would you help me to carry the stone?" Pink Floyd

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Rainer Hermanns-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Musachy,

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.

Rainer

> showcase seems fine. Rainer, whenever you can, get us an xwork release :)
>
> musachy
>
> On Thu, Jul 23, 2009 at 10:57 AM, Musachy Barroso<musachy@...>
> wrote:
>> I am done with the changes in xwork. Whoever can give a hand testing
>> what is in trunk, this is a good time to jump in :). If everything 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 Barroso<musachy@...>
>> 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 threads, 2s ramp-up
>>> period, 100 iterations) . If we are not going to use the new bycode
>>> stuff, then the upgrade doesn't seem that risky. Should we upgrade, do
>>> some testing, and release 2.1.8 (delay release a bit), or leave that
>>> for 2.1.9? With the bytecode stuff out the way I am inclined to just
>>> upgrade to 2.7.3 at once, and upgrade freemarker also.
>>>
>>> musachy
>>>
>>> --
>>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>>
>>
>>
>>
>> --
>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>
>
>
>
> --
> "Hey you! Would you help me to carry the stone?" Pink Floyd
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>


--
Rainer Hermanns
aixcept
Willibrordstraße 82
52134 Herzogenrath - Germany
w: http://aixcept.de/
t: +49 - 2406 - 979 22 11
f: +49 - 2406 - 979 22 13
m: +49 - 170 - 343 29 12

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Lukasz Lenart :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

2009/7/29 Rainer Hermanns <hermanns@...>:
> 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, but
right now I want to prepare some test application with custom
TextProvider and see how is it work.


Regards
--
Lukasz
http://www.lenart.org.pl/
http://dailylog.lenart.org.pl/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Musachy Barroso :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Chris, I wanted to double check on this, are you using Ognl 2.7.3
without Javassist? I get class loading errors without it. Please note
that xwork contains an embedded version of javassist by mistake.

musachy

On Mon, Jul 20, 2009 at 8:46 AM, Chris Pratt <thechrispratt@...> 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 AM, Musachy Barroso <musachy@...> wrote:
>
>> 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 Hermanns<hermanns@...>
>> 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 yet released :)
>> >
>> > cheers,
>> > Rainer
>> >
>> >> Sounds good. It gives rainer some more time to get xwork released.
>> >>
>> >> On 7/18/09, Dale Newfield <dale@...> wrote:
>> >>> Musachy Barroso <musachy@...> 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-unsubscribe@...
>> >>> For additional commands, e-mail: dev-help@...
>> >>>
>> >>>
>> >>
>> >>
>> >> --
>> >> Wes Wannemacher
>> >> Author - Struts 2 In Practice
>> >> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and more
>> >> http://www.manning.com/wannemacher
>> >>
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: dev-unsubscribe@...
>> >> For additional commands, e-mail: dev-help@...
>> >>
>> >>
>> >
>> >
>> > --
>> > Rainer Hermanns
>> > aixcept
>> > Willibrordstraße 82
>> > 52134 Herzogenrath - Germany
>> > w: http://aixcept.de/
>> > t: +49 - 2406 - 979 22 11
>> > f: +49 - 2406 - 979 22 13
>> > m: +49 - 170 - 343 29 12
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: dev-unsubscribe@...
>> > For additional commands, e-mail: dev-help@...
>> >
>> >
>>
>>
>>
>> --
>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@...
>> For additional commands, e-mail: dev-help@...
>>
>>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...


Re: ognl 2.7.3 performance

by Chris Pratt :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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 <musachy@...> wrote:

> Chris, I wanted to double check on this, are you using Ognl 2.7.3
> without Javassist? I get class loading errors without it. Please note
> that xwork contains an embedded version of javassist by mistake.
>
> musachy
>
> On Mon, Jul 20, 2009 at 8:46 AM, Chris Pratt <thechrispratt@...>
> 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 AM, Musachy Barroso <musachy@...>
> wrote:
> >
> >> 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 Hermanns<hermanns@...>
> >> 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 yet released :)
> >> >
> >> > cheers,
> >> > Rainer
> >> >
> >> >> Sounds good. It gives rainer some more time to get xwork released.
> >> >>
> >> >> On 7/18/09, Dale Newfield <dale@...> wrote:
> >> >>> Musachy Barroso <musachy@...> 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-unsubscribe@...
> >> >>> For additional commands, e-mail: dev-help@...
> >> >>>
> >> >>>
> >> >>
> >> >>
> >> >> --
> >> >> Wes Wannemacher
> >> >> Author - Struts 2 In Practice
> >> >> Includes coverage of Struts 2.1, Spring, JPA, JQuery, Sitemesh and
> more
> >> >> http://www.manning.com/wannemacher
> >> >>
> >> >> ---------------------------------------------------------------------
> >> >> To unsubscribe, e-mail: dev-unsubscribe@...
> >> >> For additional commands, e-mail: dev-help@...
> >> >>
> >> >>
> >> >
> >> >
> >> > --
> >> > Rainer Hermanns
> >> > aixcept
> >> > Willibrordstraße 82
> >> > 52134 Herzogenrath - Germany
> >> > w: http://aixcept.de/
> >> > t: +49 - 2406 - 979 22 11
> >> > f: +49 - 2406 - 979 22 13
> >> > m: +49 - 170 - 343 29 12
> >> >
> >> > ---------------------------------------------------------------------
> >> > To unsubscribe, e-mail: dev-unsubscribe@...
> >> > For additional commands, e-mail: dev-help@...
> >> >
> >> >
> >>
> >>
> >>
> >> --
> >> "Hey you! Would you help me to carry the stone?" Pink Floyd
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@...
> >> For additional commands, e-mail: dev-help@...
> >>
> >>
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@...
> For additional commands, e-mail: dev-help@...
>
>
< Prev | 1 - 2 - 3 | Next >