You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by David Blevins <da...@gmail.com> on 2018/10/15 19:40:05 UTC

Cross posting on lists? ( thank you Thomas -- Re: [perf] cache observers in NotificationManager#resolveObservers)

Thank you so much, Thomas, for taking a lead on the TomEE 8 startup speed!

Forwarding your notes to dev@tomee so everyone can see the hard work you're putting in.

Hey, all, what do we think about cross-posting?  If someone is doing awesome TomEE work in other communities, say BVal, OpenWebBeans, MyFaces, etc, should we encourage them to also add dev@tomee in the Cc?  Other communities may not want our noise, but seems we could easily say "we're ok being in Cc for work in your community that in some way relates to TomEE.  Go ahead and cc dev@tomee, we're cool with it.  We even like it as it helps us better know the communities behind the code in TomEE."

Seems it could help unite us a bit more.

If we like the idea, we could just add a document to the website that says "go ahead, we don't mind. we like it."  Then people could do it without worrying that it might be bad etiquette.

Thoughts?


-- 
David Blevins
http://twitter.com/dblevins
http://www.tomitribe.com

> On Oct 12, 2018, at 7:15 AM, Thomas Andraschko <an...@gmail.com> wrote:
> 
> Hi,
> 
> as you already may know from the TomEE/BVal mailing list, i'm not happy
> with the startup performance since TomEE8.
> 
> TomEE7 OWB startup ~ 4,5seconds
> TomEE8 OWB startup ~ 9seconds
> 
> i finally managed to improve the performance on 2 places.
> 
> 1) Filter framework classes in BVal on PAT
>    Reduces the startup to ~8seconds
> 
> 2) Cache obervers for systemEvents in NotificationManager#resolveObservers
>    Reduces the starup to ~7 seconds
>    Patch:
> https://gist.github.com/tandraschko/de4c72c22e65aaefe041af5a15d110e3
> 
> Do you think the patch is ok or could it break something?
> I would be happy if mark or romain could improve it :D
> 
> Best regards,
> Thomas


Re: Cross posting on lists? ( thank you Thomas -- Re: [perf] cache observers in NotificationManager#resolveObservers)

Posted by David Blevins <da...@gmail.com>.
That's awesome, Thomas!  Thank you for the note.

Looks like JL upgraded it.  I've updated the JIRA ticket type to "dependency upgrade" as we have a tool that generates release notes and puts them all together in one section.


-- 
David Blevins
http://twitter.com/dblevins
http://www.tomitribe.com

> On Nov 12, 2018, at 4:01 AM, Thomas Andraschko <an...@gmail.com> wrote:
> 
> JFYI:
> 
> we released OWB 2.0.8 today, which contains great performance improvements.
> I also created the ticket for the upgrade:
> https://issues.apache.org/jira/browse/TOMEE-2273
> 
> Am Mo., 15. Okt. 2018 um 21:47 Uhr schrieb Jonathan Gallimore <
> jonathan.gallimore@gmail.com>:
> 
>> I'm ok with cross-posting. If something isn't relevant or no-longer
>> relevant, its easy enough to drop the list that isn't relevant.
>> 
>> Jon
>> 
>> On Mon, Oct 15, 2018 at 8:40 PM David Blevins <da...@gmail.com>
>> wrote:
>> 
>>> Thank you so much, Thomas, for taking a lead on the TomEE 8 startup
>> speed!
>>> 
>>> Forwarding your notes to dev@tomee so everyone can see the hard work
>>> you're putting in.
>>> 
>>> Hey, all, what do we think about cross-posting?  If someone is doing
>>> awesome TomEE work in other communities, say BVal, OpenWebBeans, MyFaces,
>>> etc, should we encourage them to also add dev@tomee in the Cc?  Other
>>> communities may not want our noise, but seems we could easily say "we're
>> ok
>>> being in Cc for work in your community that in some way relates to TomEE.
>>> Go ahead and cc dev@tomee, we're cool with it.  We even like it as it
>>> helps us better know the communities behind the code in TomEE."
>>> 
>>> Seems it could help unite us a bit more.
>>> 
>>> If we like the idea, we could just add a document to the website that
>> says
>>> "go ahead, we don't mind. we like it."  Then people could do it without
>>> worrying that it might be bad etiquette.
>>> 
>>> Thoughts?
>>> 
>>> 
>>> --
>>> David Blevins
>>> http://twitter.com/dblevins
>>> http://www.tomitribe.com
>>> 
>>>> On Oct 12, 2018, at 7:15 AM, Thomas Andraschko <
>>> andraschko.thomas@gmail.com> wrote:
>>>> 
>>>> Hi,
>>>> 
>>>> as you already may know from the TomEE/BVal mailing list, i'm not happy
>>>> with the startup performance since TomEE8.
>>>> 
>>>> TomEE7 OWB startup ~ 4,5seconds
>>>> TomEE8 OWB startup ~ 9seconds
>>>> 
>>>> i finally managed to improve the performance on 2 places.
>>>> 
>>>> 1) Filter framework classes in BVal on PAT
>>>>   Reduces the startup to ~8seconds
>>>> 
>>>> 2) Cache obervers for systemEvents in
>>> NotificationManager#resolveObservers
>>>>   Reduces the starup to ~7 seconds
>>>>   Patch:
>>>> https://gist.github.com/tandraschko/de4c72c22e65aaefe041af5a15d110e3
>>>> 
>>>> Do you think the patch is ok or could it break something?
>>>> I would be happy if mark or romain could improve it :D
>>>> 
>>>> Best regards,
>>>> Thomas
>>> 
>>> 
>> 


Re: Cross posting on lists? ( thank you Thomas -- Re: [perf] cache observers in NotificationManager#resolveObservers)

Posted by Thomas Andraschko <an...@gmail.com>.
JFYI:

we released OWB 2.0.8 today, which contains great performance improvements.
I also created the ticket for the upgrade:
https://issues.apache.org/jira/browse/TOMEE-2273

Am Mo., 15. Okt. 2018 um 21:47 Uhr schrieb Jonathan Gallimore <
jonathan.gallimore@gmail.com>:

> I'm ok with cross-posting. If something isn't relevant or no-longer
> relevant, its easy enough to drop the list that isn't relevant.
>
> Jon
>
> On Mon, Oct 15, 2018 at 8:40 PM David Blevins <da...@gmail.com>
> wrote:
>
> > Thank you so much, Thomas, for taking a lead on the TomEE 8 startup
> speed!
> >
> > Forwarding your notes to dev@tomee so everyone can see the hard work
> > you're putting in.
> >
> > Hey, all, what do we think about cross-posting?  If someone is doing
> > awesome TomEE work in other communities, say BVal, OpenWebBeans, MyFaces,
> > etc, should we encourage them to also add dev@tomee in the Cc?  Other
> > communities may not want our noise, but seems we could easily say "we're
> ok
> > being in Cc for work in your community that in some way relates to TomEE.
> > Go ahead and cc dev@tomee, we're cool with it.  We even like it as it
> > helps us better know the communities behind the code in TomEE."
> >
> > Seems it could help unite us a bit more.
> >
> > If we like the idea, we could just add a document to the website that
> says
> > "go ahead, we don't mind. we like it."  Then people could do it without
> > worrying that it might be bad etiquette.
> >
> > Thoughts?
> >
> >
> > --
> > David Blevins
> > http://twitter.com/dblevins
> > http://www.tomitribe.com
> >
> > > On Oct 12, 2018, at 7:15 AM, Thomas Andraschko <
> > andraschko.thomas@gmail.com> wrote:
> > >
> > > Hi,
> > >
> > > as you already may know from the TomEE/BVal mailing list, i'm not happy
> > > with the startup performance since TomEE8.
> > >
> > > TomEE7 OWB startup ~ 4,5seconds
> > > TomEE8 OWB startup ~ 9seconds
> > >
> > > i finally managed to improve the performance on 2 places.
> > >
> > > 1) Filter framework classes in BVal on PAT
> > >    Reduces the startup to ~8seconds
> > >
> > > 2) Cache obervers for systemEvents in
> > NotificationManager#resolveObservers
> > >    Reduces the starup to ~7 seconds
> > >    Patch:
> > > https://gist.github.com/tandraschko/de4c72c22e65aaefe041af5a15d110e3
> > >
> > > Do you think the patch is ok or could it break something?
> > > I would be happy if mark or romain could improve it :D
> > >
> > > Best regards,
> > > Thomas
> >
> >
>

Re: Cross posting on lists? ( thank you Thomas -- Re: [perf] cache observers in NotificationManager#resolveObservers)

Posted by Jonathan Gallimore <jo...@gmail.com>.
I'm ok with cross-posting. If something isn't relevant or no-longer
relevant, its easy enough to drop the list that isn't relevant.

Jon

On Mon, Oct 15, 2018 at 8:40 PM David Blevins <da...@gmail.com>
wrote:

> Thank you so much, Thomas, for taking a lead on the TomEE 8 startup speed!
>
> Forwarding your notes to dev@tomee so everyone can see the hard work
> you're putting in.
>
> Hey, all, what do we think about cross-posting?  If someone is doing
> awesome TomEE work in other communities, say BVal, OpenWebBeans, MyFaces,
> etc, should we encourage them to also add dev@tomee in the Cc?  Other
> communities may not want our noise, but seems we could easily say "we're ok
> being in Cc for work in your community that in some way relates to TomEE.
> Go ahead and cc dev@tomee, we're cool with it.  We even like it as it
> helps us better know the communities behind the code in TomEE."
>
> Seems it could help unite us a bit more.
>
> If we like the idea, we could just add a document to the website that says
> "go ahead, we don't mind. we like it."  Then people could do it without
> worrying that it might be bad etiquette.
>
> Thoughts?
>
>
> --
> David Blevins
> http://twitter.com/dblevins
> http://www.tomitribe.com
>
> > On Oct 12, 2018, at 7:15 AM, Thomas Andraschko <
> andraschko.thomas@gmail.com> wrote:
> >
> > Hi,
> >
> > as you already may know from the TomEE/BVal mailing list, i'm not happy
> > with the startup performance since TomEE8.
> >
> > TomEE7 OWB startup ~ 4,5seconds
> > TomEE8 OWB startup ~ 9seconds
> >
> > i finally managed to improve the performance on 2 places.
> >
> > 1) Filter framework classes in BVal on PAT
> >    Reduces the startup to ~8seconds
> >
> > 2) Cache obervers for systemEvents in
> NotificationManager#resolveObservers
> >    Reduces the starup to ~7 seconds
> >    Patch:
> > https://gist.github.com/tandraschko/de4c72c22e65aaefe041af5a15d110e3
> >
> > Do you think the patch is ok or could it break something?
> > I would be happy if mark or romain could improve it :D
> >
> > Best regards,
> > Thomas
>
>